[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