[PATCH] D99926: [GVN] Introduce loop load PRE

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Just for context, I'd explored a very similar transform before in https://reviews.llvm.org/D7061.  One really key difference between the prior attempt and this one is that previously I hadn't explicitly handled loops and instead tried to match this from the original IR.  I don't think loop info was available at the time.  Though, looking at the current code, it looks like LoopInfo is optional for the pas even now.


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

https://reviews.llvm.org/D99926