[PATCH] D46451: [asan] Add instrumentation support for Myriad

[Walter Lee via Phabricator via llvm-commits]

waltl added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp:530
+                               (kMyriadMemorySize32 >> Mapping.Scale));
+      Mapping.Offset = ShadowOffset - (kMyriadMemoryOffset32 >> Mapping.Scale);
+    }
----------------
alekseyshl wrote:
> What's " - (kMyriadMemoryOffset32 >> Mapping.Scale)" part for?
It's to account for the non-zero starting DRAM address.  See https://docs.google.com/document/d/1oxmk0xUojybDaQDAuTEVpHVMi5xQX74cJPyMJbaSaRM


Repository:
  rL LLVM

https://reviews.llvm.org/D46451