[PATCH] D31730: [LCSSA] Remove redundant check

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 5 16:30:40 PDT 2017


davide abandoned this revision.
davide added a comment.

In https://reviews.llvm.org/D31730#719604, @mzolotukhin wrote:

> Can you verify that by inserting an assertion there?
>
> One thing that I'm not 100% sure there is that we might be looking at different loops in `formLCSSA` and `formLCSSAForInstructions`. In the latter we might look at subloops as far as I can tell.
>
> Michael


Nevermind, I read the code again and I realized we actually can call that on subloops.


https://reviews.llvm.org/D31730





More information about the llvm-commits mailing list