[PATCH] D36679: [InstCombine] Added support for: trunc(ashr(mul(sext(...), sext(...))) -> ashr(mul(...))

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 11:55:15 PDT 2017


spatel added a comment.

I think all of the non-multiply tests are simplified after https://reviews.llvm.org/rL310942. Can you double-check that? If so, you can remove those tests from this patch. Feel free to add them as an independent NFC commit if they cover some case that isn't covered already.

It may also be possible to simplify the code in this patch now, but I didn't check that.


https://reviews.llvm.org/D36679





More information about the llvm-commits mailing list