[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