[PATCH] D16486: Strip local symbols when using externalized debug info.

[Owen Anderson via llvm-commits]

resistor created this revision.
resistor added a reviewer: beanz.
resistor added a subscriber: llvm-commits.
resistor set the repository for this revision to rL LLVM.

When we build LLVM with externalized debug info, all debugging and symbolication related data is extracted into dSYM files prior to stripping.  As such, there is no need to preserve local symbols in LLVM binaries after dSYM creation.

This shrinks libLLVM.dylib from 58MB to 55MB on my system.

Repository:
  rL LLVM

http://reviews.llvm.org/D16486

Files:
  cmake/modules/AddLLVM.cmake

Index: cmake/modules/AddLLVM.cmake
===================================================================
--- cmake/modules/AddLLVM.cmake
+++ cmake/modules/AddLLVM.cmake
@@ -1191,7 +1191,7 @@
     endif()
     add_custom_command(TARGET ${name} POST_BUILD
       COMMAND xcrun dsymutil $<TARGET_FILE:${name}>
-      COMMAND xcrun strip -Sl $<TARGET_FILE:${name}>)
+      COMMAND xcrun strip -Sxl $<TARGET_FILE:${name}>)
   else()
     message(FATAL_ERROR "LLVM_EXTERNALIZE_DEBUGINFO isn't implemented for non-darwin platforms!")
   endif()


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16486.45740.patch
Type: text/x-patch
Size: 536 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160122/6cdcd67a/attachment.bin>