[PATCH] D40883: [LV] Ignore the cost of values that will not appear in the vectorized loop

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 02:23:48 PST 2017


fhahn added a comment.

The comment next to the VecValuesToIgnore declaration explicitly states that those instructions should be skipped during cost modelling if VF > 1 (`/// Values to ignore in the cost model when VF > 1.`)



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6869
 
+      if (VF > 1 && VecValuesToIgnore.count(&I)){
+        continue;
----------------
No curly braces here. I think this could even be merged with the previous check, as they both skip ignored values.


https://reviews.llvm.org/D40883





More information about the llvm-commits mailing list