[PATCH] D136164: [MemorySSA] Use BatchAA for clobber walker

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 07:16:51 PST 2022


nikic updated this revision to Diff 480099.
nikic marked 5 inline comments as done.
nikic edited the summary of this revision.
nikic added a comment.

Use member in ClobberWalker, and add wrapper APIs that don't require BatchAAResults (and just create it internally). Something I previously missed was updating the unit tests, and creating the BatchAA instances there does get pretty annoying for no benefit.

This patch now only ensures that individual queries use BatchAA, but doesn't yet share BatchAA across multiple queries anywhere. I'll land that part separately.


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

https://reviews.llvm.org/D136164

Files:
  llvm/include/llvm/Analysis/MemorySSA.h
  llvm/lib/Analysis/MemorySSA.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136164.480099.patch
Type: text/x-patch
Size: 20732 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221205/5c4abd95/attachment.bin>


More information about the llvm-commits mailing list