Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D151713

[ASAN] Support asan for masked.expandload/compressstore.
ClosedPublic
Actions

Authored by fakepaper56 on May 30 2023, 7:59 AM.

Details

Reviewers
reames
MaskRay
craig.topper
kito.cheng
Commits
rGf79ad31f177d: [ASAN] Support asan for masked.expandload/compressstore.
Summary

The patch uses a way similiar to vp.load/store and consider the mask popcount as
the effetive vector length.

Diff Detail

Event Timeline

fakepaper56 created this revision.May 30 2023, 7:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 30 2023, 7:59 AM
Herald added subscribers: Enna1, hiraditya. · View Herald Transcript
fakepaper56 requested review of this revision.May 30 2023, 7:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 30 2023, 7:59 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript
Harbormaster completed remote builds in B235298: Diff 526624.May 30 2023, 8:50 AM
fakepaper56 updated this revision to Diff 530503.Jun 12 2023, 7:31 AM

Rebase and ping.

Harbormaster completed remote builds in B238185: Diff 530503.Jun 12 2023, 8:13 AM
fakepaper56 updated this revision to Diff 538893.Jul 10 2023, 7:30 PM

Rebase and ping.

Herald added a subscriber: wangpc. · View Herald TranscriptJul 10 2023, 7:30 PM
craig.topper accepted this revision.Jul 10 2023, 8:06 PM

LGTM

llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
1369

effective*

This revision is now accepted and ready to land.Jul 10 2023, 8:06 PM
fakepaper56 updated this revision to Diff 538899.Jul 10 2023, 8:07 PM

Fix typo.

fakepaper56 marked an inline comment as done.Jul 10 2023, 8:08 PM
Harbormaster completed remote builds in B244331: Diff 538899.Jul 10 2023, 9:31 PM
This revision was landed with ongoing or failed builds.Jul 10 2023, 11:41 PM
Closed by commit rGf79ad31f177d: [ASAN] Support asan for masked.expandload/compressstore. (authored by fakepaper56). · Explain Why
This revision was automatically updated to reflect the committed changes.
fakepaper56 added a commit: rGf79ad31f177d: [ASAN] Support asan for masked.expandload/compressstore..

Revision Contents
Changeset List

PathSize
llvm/
lib/
Transforms/
Instrumentation/
23 lines
test/
Instrumentation/
AddressSanitizer/
79 lines

Diff 538925

View Options

llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp

Loading...
View Options

llvm/test/Instrumentation/AddressSanitizer/asan-masked-load-store.ll

Loading...