[PATCH] D62938: [SLP] Forbid to vectorize bundles with same opcode but different IR flags
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 6 14:19:28 PDT 2019
RKSimon 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.
@dtemirbulatov It might be worth adding tests that check that we do retain the common subset?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62938/new/
https://reviews.llvm.org/D62938
More information about the llvm-commits
mailing list