[PATCH] D84873: AMDGPU: In determining load clobbering in AnnotateUniform, don't scan if there are too many blocks.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 12:30:10 PDT 2020


asbirlea added a comment.

In D84873#2182544 <https://reviews.llvm.org/D84873#2182544>, @arsenm wrote:

> Does MemorySSA have the same problem? Could we just switch this to use MemorySSA?

Disclaimer: I know nothing about this pass or the purpose of this patch, just trying to answer this question.
MemorySSA has its own internal threshold limiting the number of **memory instructions** that are traversed upwards. It does not care at how many blocks those memory instructions are spread over.


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

https://reviews.llvm.org/D84873



More information about the llvm-commits mailing list