[PATCH] Working on reconciling out-of-tree patches to compiler-rt for building for iOS.

Alexey Samsonov vonosmas at gmail.com
Tue Jun 23 11:07:29 PDT 2015


================
Comment at: lib/asan/asan_mac.cc:27
@@ -26,2 +26,3 @@
 
-#include <crt_externs.h>  // for _NSGetArgv
+#if !TARGET_OS_IPHONE
+#include <crt_externs.h>  // for _NSGetArgv and _NSGetEnviron
----------------
`SANITIZER_IOS`

================
Comment at: lib/asan/asan_mac.cc:73
@@ -64,2 +72,3 @@
 void LeakyResetEnv(const char *name, const char *name_value) {
+#if !TARGET_OS_IPHONE
   char ***env_ptr = _NSGetEnviron();
----------------
Looks like this logic is copied from http://reviews.llvm.org/D10510. Consider introducing a function instead.

================
Comment at: lib/asan/asan_mapping.h:101
@@ -100,3 +100,3 @@
 static const u64 kDefaultShadowOffset32 = 1ULL << 29;  // 0x20000000
 static const u64 kIosShadowOffset32 = 1ULL << 30;  // 0x40000000
 static const u64 kDefaultShadowOffset64 = 1ULL << 44;
----------------
Please group iossim constants together,

================
Comment at: lib/asan/asan_mapping.h:120
@@ +119,3 @@
+# if SANITIZER_WORDSIZE == 32
+#   if SANITIZER_IOSSIM
+#     define SHADOW_OFFSET kIosSimShadowOffset32
----------------
indentation looks inconsistent.

http://reviews.llvm.org/D10515

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list