[PATCH] D107786: [AIX]: Fix option processing for -b

Ettore Tiotto via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 9 16:45:26 PDT 2021


etiotto added inline comments.


================
Comment at: clang/test/Driver/Xlinker-args.c:26
 // LINUX: "--no-demangle" "-e" "_start" "one" "two" "three" "four" "-z" "five" "-r" {{.*}} "-T" "a.lds"
-// AIX: "-b" "one" 
+// AIX: "-b" "one" "-b" "two"
 // NOT-AIX: error: unsupported option '-b' for target 'powerpc-unknown-linux'
----------------
jsji wrote:
> It would be better if we can tested mixing other options like `-z` .
I think the test is adequate as it guarantees options passed via "-b" are not going to be duplicated (which is the problem this PR addresses). 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107786



More information about the cfe-commits mailing list