[PATCH] [OPENMP][LV] Respect Hints.Force meta-data for loops in LoopVectorizer
tnowicki at apple.com
tnowicki at apple.com
Mon Apr 28 11:55:08 PDT 2014
Hi Zinovy,
Sorry for the delay.
LGTM. Just a few small things.
+ // Check the loop for a trip count threshold:
+ // do not vectorize loops with a tiny trip count.
+ {
+ BasicBlock *Latch = L->getLoopLatch();
Could you remove the extra braces?
+ if (ForceVectorization && VF > 1) {
+ Width = 2;
+ Cost = expectedCost(Width) / (float)Width;
+ }
Could you add a comment to explain that scalar width is not considered because the user wants vectorization.
Thanks for working on this,
Tyler
http://reviews.llvm.org/D3423
More information about the llvm-commits
mailing list