[PATCH] D84608: [LAA] Avoid adding pointers to the checks if they are not needed.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 11:20:24 PDT 2020


fhahn added a comment.

In D84608#2174659 <https://reviews.llvm.org/D84608#2174659>, @baziotis wrote:

> Instead of accumulating in `AccessInfos`, couldn't we just reiterate the alias set (if we didn't bail out)?


Yes, I think either approach is fine. The current way avoids a few repeated lookups, but in the grand scheme of things I don't think it makes a big difference.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84608





More information about the llvm-commits mailing list