yaxunl added inline comments.
================
Comment at: clang/test/Driver/compress.c:26
+// CHECK-OPT_GZ_EQ_NONE: {{".*clang.*".* "--compress-debug-sections=none"}}
+// CHECK-OPT_GZ_EQ_NONE: {{".*ld.*".* "--compress-debug-sections=none"}}
+
----------------
MaskRay wrote:
> If the user sets CLANG_DEFAULT_LINKER to a string not containing "ld", this
> line will not match. Matching a different substring on the linker command
> line can be more robust, e.g. `"-m" "elf_x86_64"`
"-m" "elf_x86_64" only works for x86_64, but we also have amdgcn target.
I will just remove "ld".
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87321/new/
https://reviews.llvm.org/D87321
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
