[PATCH] [compiler-rt] Upstreaming small changes from the Rust project.

Saleem Abdulrasool compnerd at compnerd.org
Mon Dec 8 20:12:59 PST 2014


REPOSITORY
  rL LLVM

================
Comment at: lib/builtins/assembly.h:45
@@ -42,3 +44,3 @@
 
-#else // !__APPLE__ && !__ELF__
+#elif defined(__WIN32__)
 
----------------
I'd rather leave this part of the change out.  We don't have a good way to test if the target is COFF, which these directives target, not specifically Windows.

================
Comment at: make/config.mk:47
@@ -46,3 +46,3 @@
 COMMON_INCLUDES=-I${ProjSrcRoot}/lib -I${ProjSrcRoot}/include
-COMMON_CXXFLAGS=-std=c++11 -fno-exceptions -fPIC -funwind-tables $(COMMON_INCLUDES)
-COMMON_CFLAGS=-fPIC $(COMMON_INCLUDES)
+COMMON_CXXFLAGS=-std=c++11 -fno-exceptions -funwind-tables $(COMMON_INCLUDES)
+COMMON_CFLAGS=$(COMMON_INCLUDES)
----------------
Whats the reason for sinking the -fPIC into the individual makefiles rather than keeping it here?

http://reviews.llvm.org/D6547






More information about the llvm-commits mailing list