The only reason we export symbols from these tools is to support plugins; if we don't have plugins, exporting symbols just bloats the executable and makes LTO less effective.
See review of D75879 for the discussion that led to this patch.
Paths
| Differential D76527
Don't export symbols from clang/opt/llc if plugins are disabled. ClosedPublic Authored by efriedma on Mar 20 2020, 2:08 PM.
Details Summary The only reason we export symbols from these tools is to support plugins; if we don't have plugins, exporting symbols just bloats the executable and makes LTO less effective. See review of D75879 for the discussion that led to this patch.
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Mar 20 2020, 2:35 PM Closed by commit rG896335bfb8ea: Don't export symbols from clang/opt/llc if plugins are disabled. (authored by efriedma). · Explain WhyMar 23 2020, 12:33 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 251757 clang/tools/driver/CMakeLists.txt
llvm/cmake/modules/AddLLVM.cmake
llvm/tools/bugpoint/CMakeLists.txt
llvm/tools/llc/CMakeLists.txt
llvm/tools/llvm-stress/CMakeLists.txt
llvm/tools/opt/CMakeLists.txt
llvm/unittests/Passes/CMakeLists.txt
|