[PATCH] D107162: [lit] Have REQUIRES support the target triple
Paul Robinson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 5 14:28:45 PDT 2021
probinson added a comment.
The two tests that went from Unsupported to Pass are
LLVM :: tools/llvm-readobj/COFF/call-graph-profile-err.s
LLVM :: tools/llvm-readobj/COFF/call-graph-profile.s
both of which have `REQUIRES: x86` which matches the default triple on my system (x86_64-unknown-linux) but isn't the right way to constrain the test in a pre-patch world; that would be `target-x86` or `target-x86_64` or both.
Reviewers have expressed some need for caution in applying this patch, and there might well be tests out there that mistakenly have things like `REQUIRES: arm` or `REQUIRES: powerpc` and don't happily pass once they are suddenly enabled for the first time. So, I guess that means I won't be pushing this upstream at quitting time on Friday after all. :-)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107162/new/
https://reviews.llvm.org/D107162
More information about the llvm-commits
mailing list