[PATCH] Partial fix for bug 22589

hfinkel at anl.gov hfinkel at anl.gov
Tue Feb 17 20:39:50 PST 2015


Thanks for working on this.

I think this makes sense as a general simplification. Previously, we were trying to compare TripCount to Count, which required an additional check that TripCount == BECount+1 did not overflow. Comparing BECount to Count-1 is better because we don't need a separate overflow check.


================
Comment at: lib/Transforms/Utils/LoopUnrollRuntime.cpp:346
@@ -336,1 +345,3 @@
+  Value *BECount = Expander.expandCodeFor(BECountSC, BECountSC->getType(),
+                                             PreHeaderBR);
 
----------------
Extra indentation.

http://reviews.llvm.org/D7715

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list