[PATCH] [nolibc] Add RTSanitizerCommonNoLibc, a libc-free subset of sanitizer_common.

Kostya Serebryany kcc at google.com
Wed May 8 01:39:27 PDT 2013


  I still don't like this ifdef clatter.
  Maybe we could create a few new files, like sanitizer_xxx_libcdep.cc, which and only which are allowed to depend on libc?
  Then your target will simply omit those files.


================
Comment at: lib/sanitizer_common/sanitizer_allocator.cc:33
@@ -32,1 +32,3 @@
 
+#if !SANITIZER_NOLIBC
+
----------------
Please give us a chance to get rid of this. The change is under review now. 
Unfortunately it may still take some time since samsonov@ is on vacation. :( 

================
Comment at: lib/sanitizer_common/sanitizer_linux.cc:627
@@ -617,3 +626,3 @@
 
-#ifndef SANITIZER_GO
+#ifndef SANITIZER_NOLIBC
 //------------------------- SlowUnwindStack -----------------------------------
----------------
Hm? replaced GO with NOLIBC ? 


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



More information about the llvm-commits mailing list