[PATCH] D69563: [LV] Strip wrap flags from vectorized reductions

[Hideki Saito via Phabricator via llvm-commits]

hsaito added a comment.

In D69563#1748967 <https://reviews.llvm.org/D69563#1748967>, @dantrushin wrote:

> @hsaito: What do you think?


Checked with a few of my colleagues. They say

1. SCEV drops a lot of no-wrap flags because of inherent reassociation issue in design
2. InstCombine and scalar reassociate passes propagate the wrap[ flags if they can prove the flags are still valid.

So, dropping is the right thing to do. Thanks for raising the issue and fixing it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69563/new/

https://reviews.llvm.org/D69563