[PATCH] D146208: [ASAN] Support memory checks on vp.load/store.

[Yeting Kuo via Phabricator via llvm-commits]

fakepaper56 added inline comments.


================
Comment at: llvm/test/Instrumentation/AddressSanitizer/asan-vp-load-store.ll:29
+; CHECK-NEXT:    [[TMP8:%.*]] = ptrtoint ptr [[TMP7]] to i64
+; CHECK-NEXT:    call void @__asan_storeN(i64 [[TMP8]], i64 4)
+; CHECK-NEXT:    br label [[TMP9]]
----------------
kito-cheng wrote:
> Could you take a look to see why it use `__asan_storeN`, not `__asan_store4`? I saw D145198 can just generate `__asan_store4`?
`__asan_load4` needs the input address is 4-bytes aligned. I will refine the test to check that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146208/new/

https://reviews.llvm.org/D146208