[PATCH] D67076: [LoopInterchange] Make sure we create PHI nodes for uses in split off latch.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 12:16:52 PDT 2019


efriedma added a comment.

Is there a potential correctness issue here, if some operation that isn't the exit condition or the IV increment somehow ends up in the new latch?

It seems more straightforward to ensure the new latch contains exactly the instructions you expect, rather than creating an LCSSA PHI for a value you've proved is loop-invariant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67076





More information about the llvm-commits mailing list