[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