[PATCH] D71250: [LV] Create new vector loop preheader so it contains vectorizer generated code only.
Renato Golin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 10 11:05:01 PST 2019
rengolin added a comment.
This change introduces a number of new branches in the code. While not necessarily the hottest code, it may prove significant on short loops, especially nested loops. Without evidence in the form of benchmarks etc. it's hard to justify.
Furthermore, I'm not sure I agree with the statement that this makes the analysis somehow simpler. Do you have an explicit goal in mind?
================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:2680
+ // No trip count check needed for predicated vectorization.
+ if (Cost->foldTailByMasking())
+ return;
----------------
Are you sure this is semantically valid on all targets?
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