[PATCH] D71250: [LV] Create new vector loop preheader so it contains vectorizer generated code only.

Evgeniy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 21:10:47 PST 2019


ebrevnov marked an inline comment as done.
ebrevnov added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:2680
+  // No trip count check needed for predicated vectorization.
+  if (Cost->foldTailByMasking())
+    return;
----------------
rengolin wrote:
> Are you sure this is semantically valid on all targets?
That doesn't change existing semantics. It simply allows not to generate trivially dead code because otherwise there is an unconditional branch to vector loop.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71250/new/

https://reviews.llvm.org/D71250





More information about the llvm-commits mailing list