[PATCH] D88566: be more specific when testing for no fuse-ld warnings

Ties Stuij via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 7 10:21:49 PDT 2020


stuij added a comment.

Hi @MaskRay. Yes, so we're seeing a warning specific to our Armcompiler toolchain, so I'm guessing that isn't relevant to OSS LLVM:
`armclang: warning: '--target=x86_64-unknown-linux' is not supported.`

As David Green pointed out, we have a perfectly fine workaround. But I figured that a similar situation might crop up in OSS LLVM, and this way the test is a bit more future proof, and we might spare some future head-scratching.

However I feel bad already for wasting our time with such a minor change. Feel free to reject it :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88566/new/

https://reviews.llvm.org/D88566



More information about the cfe-commits mailing list