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

Yeting Kuo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 02:22:03 PDT 2023


fakepaper56 updated this revision to Diff 515260.
fakepaper56 added a comment.

This update does,

1. Only check whether EVL is zero when EVL is not nullptr. It helps not generate reduduent zero check for non-EVL InterestingMemoryOperand.
2. Clean up my patch.

I am sorry for that the quality of the patch was terrible. I guess it was due to
that I was too rushed to create the revision before my military educational
recall. It's reasonable that no one wanted to reveiw the terrible patch.
I promise that I will improve the quality of my succeeding patches in LLVM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146208

Files:
  llvm/include/llvm/Transforms/Instrumentation/AddressSanitizerCommon.h
  llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h
  llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
  llvm/lib/Transforms/Utils/BasicBlockUtils.cpp
  llvm/test/Instrumentation/AddressSanitizer/asan-vp-load-store.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D146208.515260.patch
Type: text/x-patch
Size: 28379 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230420/0e6dd251/attachment-0001.bin>


More information about the llvm-commits mailing list