[PATCH] D79078: [VectorCombine] Leave reduction operation to SLP
JunMa via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 12 06:24:53 PDT 2020
junparser added a comment.
hi @lebedev.ri , sorry for the late response.
I have checked the reduction tree build in SLPVectorizer, for now it only support same operation. It seems we can revert the transformation when match reduction operation in matchAssociativeReduction. However, I don't think it is a good idea.
Also the extra phase-ordering test shows that this patch has minimal impact. Except the first case which is vectorized by SLP, the other cases keep the same IR as before.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79078/new/
https://reviews.llvm.org/D79078
More information about the llvm-commits
mailing list