[PATCH] D126939: [SLP] Avoid converting undef to poison when gathering.
Vasileios Porpodas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 2 19:51:19 PDT 2022
vporpo added a comment.
Nuno was right, the previous patch was over reduced. The issue happens when the input is reachable and undef. I think in this case SLP is actually changing the program behavior.
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7559
NumValues = VF;
+ } else if (UniqueValues.size() == 1 && NumValues == 1) {
+ NumValues = VF;
----------------
@ABataev I am not sure this is the right fix for it, please take a look.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126939/new/
https://reviews.llvm.org/D126939
More information about the llvm-commits
mailing list