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

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 12 15:37:31 PDT 2020


hubert.reinterpretcast marked an inline comment as done.
hubert.reinterpretcast added inline comments.


================
Comment at: llvm/cmake/modules/HandleLLVMOptions.cmake:202
+    # we can limit the processing for archive members to only those that are otherwise referenced.
+    append("-bcdtors:mbr"
+           CMAKE_EXE_LINKER_FLAGS CMAKE_MODULE_LINKER_FLAGS CMAKE_SHARED_LINKER_FLAGS)
----------------
daltenty wrote:
> This seems more generally applicable on AIX. Could we promote this out of the XL specific block and just pass it as a general linker option?
I don't think we should move this out of the XL-specific block at this time. GCC does not use the same mechanism for static initialization on AIX.


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