[PATCH] D15625: Add iOS/watchOS/tvOS support for ASan (llvm part)

[Anna Zaks via llvm-commits]

zaks.anna added inline comments.

================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:365
@@ -360,1 +364,3 @@
       Mapping.Offset = kFreeBSD_ShadowOffset32;
+    else if (IsIOS && IsX86)
+      Mapping.Offset = kIOSSimShadowOffset32;
----------------
samsonov wrote:
> Looks like the order becomes important here. Maybe:
>   else if (IsIOS)
>     Mapping.Offset = IsX86 ? kIOSSimShadowOffset32 : kIOSShadowOffset32;
> here and below? And a comment describing the rationale behind this (I guess, it's just an heuristic, right?)
>(I guess, it's just an heuristic, right?)
What do you mean? We are setting the location of the shadow on the device and the simulator (IsIOS and IsX86 -> simulator). The simulator has the host's VM layout. The device has a very different (and restricted) VM layout.


http://reviews.llvm.org/D15625