[PATCH] D34373: [LV] Optimize for size when vectorizing loops with tiny trip count

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 17:25:57 PDT 2017


hfinkel added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7829
       DEBUG(dbgs() << "\n");
-      ORE->emit(createMissedAnalysis(Hints.vectorizeAnalysisPassName(),
-                                     "NotBeneficial", L)
-                << "vectorization is not beneficial "
-                   "and is not explicitly forced");
-      return false;
+      OptForSize = true;
     }
----------------
Please add some comments here explaining this behavior (it seems to make sense after reading the patch description, but without that context, I'd likely find this confusing).


https://reviews.llvm.org/D34373





More information about the llvm-commits mailing list