[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