[PATCH] D79749: [CMake][AIX] Add `-bcdtors:mbr` option when building with IBM XL

David Tenty via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 15 13:04:57 PDT 2020


daltenty added inline comments.


================
Comment at: llvm/cmake/modules/HandleLLVMOptions.cmake:200
+    # static storage duration even for archive members that are otherwise unreferenced.
+    # Since `--whole-archive` is not used by the LLVM build to keep such initializations for Linux,
+    # we can limit the processing for archive members to only those that are otherwise referenced.
----------------
`--whole-archive` is used on Linux for building the shared libraries when `-DLLVM_BUILD_LLVM_DYLIB=yes`. Do we need to be concerned about this case?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79749





More information about the llvm-commits mailing list