[PATCH] D62938: [SLP] Forbid to vectorize bundles with same opcode but different IR flags
Dinar Temirbulatov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 6 12:06:07 PDT 2019
dtemirbulatov abandoned this revision.
dtemirbulatov added a comment.
In D62938#1532286 <https://reviews.llvm.org/D62938#1532286>, @lebedev.ri wrote:
> Please can you add a better explanation of the problem to the description of the patch?
> I'm not sure what the problem is, you are allowed to drop `nuw`/`nsw` flags: https://rise4fun.com/Alive/plNm
> So the new vectorized binop should simply take the smallest common subset of flags, which likely most often means no flags.
yes, it looks now correct, abandoning.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62938/new/
https://reviews.llvm.org/D62938
More information about the llvm-commits
mailing list