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

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 25 07:44:46 PDT 2022


nikic added a comment.

Ping :)

In D136164#3865041 <https://reviews.llvm.org/D136164#3865041>, @foad wrote:

>> e.g. for NewGVN I think we should be able to use a single BatchAA instance for all queries.
>
> @kmitropoulou it sounds like this could help with NewGVN compile time.

Without looking too deeply into it, it's likely that the changes proposed by @kmitropoulou would actually prevent sharing BatchAA for all queries, because they introduce IR changes to the analysis phase, while NewGVN is generally designed to separate analysis and transformation.


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

https://reviews.llvm.org/D136164



More information about the llvm-commits mailing list