[PATCH] D70919: [Hexagon] Avoid passing unsupported options to lld when -fuse-ld=lld is used
Krzysztof Parzyszek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 16 10:34:50 PST 2019
kparzysz added inline comments.
================
Comment at: clang/test/Driver/hexagon-toolchain-elf.c:560
+// CHECK082-NOT: "-march=hexagon"
+// CHECK082-NOT: "-mcpu=hexagon"
----------------
kparzysz wrote:
> I don't think there will ever be `"-mcpu=hexagon"` (including the quotation marks). Maybe this should just check for any `-march` and `-mcpu`.
This is still wrong. Assume that we are passing `-mcpu=hexagonv60` to lld. The options will show up with quotation marks, and so it will be `"-mcpu=hexagonv60"`. The testcase is checking for `"-mcpu"` which will not match `"-mcpu=hexagonv60"`, and the testcase will pass even though it shouldn't.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70919/new/
https://reviews.llvm.org/D70919
More information about the cfe-commits
mailing list