[PATCH] D91343: [InstCombine] Optimize away the unnecessary multi-use sign-extend
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 12 13:51:35 PST 2020
spatel added a comment.
In D91343#2392480 <https://reviews.llvm.org/D91343#2392480>, @RKSimon wrote:
> Can the test cases be simplified? I'm not sure you need the multiple blocks to just check for multiuse?
Note that I posted an alternate version at D91364 <https://reviews.llvm.org/D91364> and added tests for it. Unless I missed some corner case covered by these tests, they should not be needed now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91343/new/
https://reviews.llvm.org/D91343
More information about the llvm-commits
mailing list