[PATCH] D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 22 09:05:51 PDT 2020


hubert.reinterpretcast added inline comments.


================
Comment at: clang/test/Driver/aix-ld.cpp:1
+// Check powerpc-ibm-aix7.1.0.0, 32-bit. 'bcdtors' and Arguemnt order.
+// // RUN: %clang++ -no-canonical-prefixes %s -### -o %t.o 2>&1 \
----------------
ZarkoCA wrote:
> s/Arguemnt/Argument/
Lowercase "argument".


================
Comment at: clang/test/Driver/aix-ld.cpp:2
+// Check powerpc-ibm-aix7.1.0.0, 32-bit. 'bcdtors' and Arguemnt order.
+// // RUN: %clang++ -no-canonical-prefixes %s -### -o %t.o 2>&1 \
+// // RUN:         -Wl,-bnocdtors \
----------------
Let's not use a `.o` suffix for the link step output. It seems using no `-o` is fine for tests with `-###`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80415





More information about the cfe-commits mailing list