[PATCH] D48160: [SCEV] Simplify trunc-of-add/mul to add/mul-of-trunc under more circumstances.

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 3 04:21:45 PDT 2018


mkazantsev added a comment.

Hi @jlebar

This patch seems to cause the bug https://bugs.llvm.org/show_bug.cgi?id=39160 on vectorization stage. See reproducer in the bug.

Note that I have taken a look into your patch and don't see any obvious problems in that; it is likely to expose some underlying bug rather than introduce a new one, but it's not for sure.

I'm going to investigate it tomorrow. I'd appreciate if you could take a look into it or revert the patch until its reason is clear.

Thanks,
Max


Repository:
  rL LLVM

https://reviews.llvm.org/D48160





More information about the llvm-commits mailing list