[PATCH] D77617: [AddressSanitizer] Split out memory intrinsic handling

Jann Horn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 6 18:34:06 PDT 2020


thejh created this revision.
thejh added a project: LLVM.
Herald added subscribers: llvm-commits, hiraditya.
thejh added a parent revision: D77616: [AddressSanitizer] Refactor ClDebug{Min,Max} handling.
thejh added a child revision: D77618: [AddressSanitizer] Refactor: Permit >1 interesting operands per instruction.
thejh edited the summary of this revision.
thejh added reviewers: kcc, glider.

In both AddressSanitizer and HWAddressSanitizer, we first collect
instructions whose operands should be instrumented and memory intrinsics,
then instrument them. Both during collection and when inserting
instrumentation, they are handled separately.

Collect them separately and instrument them separately. This is a bit
more straightforward, and prepares for collecting operands instead of
instructions in a future patch.

This is patch 2/4 of a patch series:
https://reviews.llvm.org/D77616 [PATCH 1/4] [AddressSanitizer] Refactor ClDebug{Min,Max} handling
https://reviews.llvm.org/D77617 [PATCH 2/4] [AddressSanitizer] Split out memory intrinsic handling
https://reviews.llvm.org/D77618 [PATCH 3/4] [AddressSanitizer] Refactor: Permit >1 interesting operands per instruction
https://reviews.llvm.org/D77619 [PATCH 4/4] [AddressSanitizer] Instrument byval call arguments


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77617

Files:
  llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
  llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77617.255556.patch
Type: text/x-patch
Size: 3573 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200407/e01b29f0/attachment.bin>


More information about the llvm-commits mailing list