[PATCH] Add a CMake option COMPILER_RT_DEBUG for building runtimes with full debug info.

Alexey Samsonov samsonov at google.com
Mon Oct 21 08:52:59 PDT 2013



================
Comment at: lib/sanitizer_common/sanitizer_linux.cc:222
@@ -222,1 +221,3 @@
+  kernel_timeval tv;
+  internal_memset(&tv, 0, sizeof(tv));
   internal_syscall(__NR_gettimeofday, &tv, 0);
----------------
Please commit this as a separate change.

================
Comment at: lib/CMakeLists.txt:199
@@ -198,1 +198,3 @@
 
+macro(pythonize_bool var)
+  if (${var})
----------------
No, I'd like to avoid this. You silently change the value of option COMPILER_RT_DEBUG, so it may have a surprising value if one decides to use it later. I think you may just compare config.compiler_rt_debug to "ON"/"OFF" strings. This approach is already used in LLVM test suite - look for config.llvm_use_intel_jitevents attribute.


http://llvm-reviews.chandlerc.com/D1984



More information about the llvm-commits mailing list