[llvm] r198528 - Add a FIXME.

Nico Weber nicolasweber at gmx.de
Sat Jan 4 16:37:46 PST 2014


Author: nico
Date: Sat Jan  4 18:37:45 2014
New Revision: 198528

URL: http://llvm.org/viewvc/llvm-project?rev=198528&view=rev
Log:
Add a FIXME.

Modified:
    llvm/trunk/cmake/modules/AddLLVM.cmake

Modified: llvm/trunk/cmake/modules/AddLLVM.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddLLVM.cmake?rev=198528&r1=198527&r2=198528&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/AddLLVM.cmake (original)
+++ llvm/trunk/cmake/modules/AddLLVM.cmake Sat Jan  4 18:37:45 2014
@@ -77,6 +77,9 @@ function(add_llvm_symbol_exports target_
 endfunction(add_llvm_symbol_exports)
 
 function(add_dead_strip target_name)
+  # FIXME: With MSVS, consider compiling with /Gy and linking with /OPT:REF?
+  # But MinSizeRel seems to add that automatically, so maybe disable these
+  # flags instead if LLVM_NO_DEAD_STRIP is set.
   if(NOT CYGWIN AND NOT MINGW AND NOT MSVC)
     if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Darwin")
        SET(CMAKE_CXX_FLAGS





More information about the llvm-commits mailing list