[PATCH] D32730: LV: Don't insert runtime ptr checks on divergent targets

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 02:46:10 PDT 2017


aemerson added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6318
+  if (Legal->getRuntimePointerChecking()->Need && TTI.hasBranchDivergence()) {
+    // TODO: It may by useful to do since it's still likely to be dynamically
+    // uniform if the target can skip.
----------------
Is it worth adding an optimization remark here to match with the rest of the checks in this function?


https://reviews.llvm.org/D32730





More information about the llvm-commits mailing list