[PATCH] D107162: [lit] Have REQUIRES support the target triple
Paul Robinson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 6 15:06:33 PDT 2021
probinson added a comment.
In D107162#2932060 <https://reviews.llvm.org/D107162#2932060>, @thakis wrote:
> Looks like adding the `not` helped. That's a bit unexpected since lld isn't supposed to exit non-0 there. Do you know how the test worked before? Did it just never run?
>
> @int3 too.
Correct, the `REQUIRES: x86` would never have matched anything so it just didn't run.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107162/new/
https://reviews.llvm.org/D107162
More information about the llvm-commits
mailing list