Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Target-independent tests should not specify a triple. #15397

Open
atrick opened this issue Jan 22, 2013 · 2 comments
Open

Target-independent tests should not specify a triple. #15397

atrick opened this issue Jan 22, 2013 · 2 comments
Labels
bugzilla Issues migrated from bugzilla llvm:codegen

Comments

@atrick
Copy link
Contributor

atrick commented Jan 22, 2013

Bugzilla Link 15025
Version trunk
OS All
Blocks llvm/llvm-bugzilla-archive#15026
Attachments List of supposedly target independent tests with a target triple.
CC @jryans,@RKSimon

Extended Description

Lit tests that require a triple should be moved into a platform directory so that they will be skipped by compilers that don't include the necessary back end. Tests that don't require a triple should not specify one so that any compiler can run them with deterministic behavior.

The attached triple-tests.lst lists the test cases that require cleanup.
@atrick
Copy link
Contributor Author

atrick commented Jan 22, 2013

The challenge here is to ensure the original intent of the test is preserved. I can certainly try to cleanup LSR tests as a visit them, but there are a lot of violations. This looks like a long-term group effort to me.

@atrick
Copy link
Contributor Author

atrick commented Nov 26, 2021

mentioned in issue llvm/llvm-bugzilla-archive#15026

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla llvm:codegen
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@atrick