[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