[PATCH] D107162: [lit] Have REQUIRES support the target triple

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 6 14:34:08 PDT 2021


thakis added a subscriber: int3.
thakis added a comment.

In D107162#2931752 <https://reviews.llvm.org/D107162#2931752>, @probinson wrote:

> In D107162#2931422 <https://reviews.llvm.org/D107162#2931422>, @thakis wrote:
>
>> Looks like this breaks check-lld on macOS: http://45.33.8.238/macm1/15677/step_10.txt
>>
>> Please take a look.
>
> Seems to be better now: http://45.33.8.238/macm1/15689/summary.html

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.


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