[PATCH] D13139: [builtins] Build both static and shared versions of the builtins library.

Vasileios Kalintiris via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 09:44:13 PDT 2015


vkalintiris added inline comments.

================
Comment at: lib/builtins/CMakeLists.txt:343
@@ +342,3 @@
+                                SOURCES ${${arch}_SOURCES}
+                                CFLAGS "-std=c99"
+                                PARENT_TARGET builtins)
----------------
Why do we use the C99 standard explicitly here? I would like to change that and use whatever is the compiler's default or make it a user defined option. At least for MIPS, this would allow us to include atomic.c in builtins, when using C11.


http://reviews.llvm.org/D13139





More information about the llvm-commits mailing list