[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