Re-apply "[CMake][compiler-rt][AArch64] Avoid preprocessing LSE builtins separately"
authorRaul Tambre <raul@tambre.ee>
Tue, 15 Dec 2020 06:15:58 +0000 (08:15 +0200)
committerRaul Tambre <raul@tambre.ee>
Tue, 15 Dec 2020 06:18:07 +0000 (08:18 +0200)
commita72d462eb9b9f5698e081c8ccf3bc7fa3e35da08
treefbf92d2c46bbffb953267509007b578761890de8
parent3000c19df64f89ff319590f3a6e4d6b93d20983d
Re-apply "[CMake][compiler-rt][AArch64] Avoid preprocessing LSE builtins separately"

aa772fc85e0f526615c78b9c3979c2be945a754c (D92530) has landed fixing relocations on Darwin.
3000c19df64f89ff319590f3a6e4d6b93d20983d (D93236) has landed working around an assembly parser bug on Darwin.
Previous quick-fix d9697c2e6b153ac7dc40a69450d9b672f71b1029 (D93198) included in this commit.

Invoking the preprocessor ourselves is fragile and would require us to replicate CMake's handling of definitions, compiler flags, etc for proper compatibility.
In my toolchain builds this notably resulted in a bunch of warnings from unused flags as my CMAKE_C_FLAGS includes CPU-specific optimization options.
Notably this part was already duplicating the logic for VISIBILITY_HIDDEN define.

Instead, symlink the files and set the proper set of defines on each.
This should also be faster as we avoid invoking the compiler multiple times.

Fixes https://llvm.org/PR48494

Differential Revision: https://reviews.llvm.org/D93278
compiler-rt/cmake/Modules/CompilerRTDarwinUtils.cmake
compiler-rt/lib/builtins/CMakeLists.txt
compiler-rt/lib/builtins/aarch64/lse.S