[PATCH] D32261: [LoopUnroll] Don't try to unroll non-rotated loops

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 12:44:53 PDT 2017


efriedma added inline comments.


================
Comment at: lib/Transforms/Utils/LoopUnroll.cpp:332
+      });
+  if (NotRotated) {
+    DEBUG(dbgs() << "  Can't unroll; loop not rotated\n");
----------------
It might be simpler to explicitly check the two successors; one should be the header, and the other outside the loop.

You also can't delete the check that BI is non-null and conditional; we specifically assume that later on.


https://reviews.llvm.org/D32261





More information about the llvm-commits mailing list