[PATCH] D63382: [InstCombine] fold a shifted zext to a select

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 12 07:42:44 PDT 2019


spatel added a comment.

I haven't tested this on trunk, but I added the DAGCombiner reversals:
rL374397 <https://reviews.llvm.org/rL374397>
rL374555 <https://reviews.llvm.org/rL374555>
...so this should be good to go. Should I commandeer?

There may still be regressions because most in-trunk targets don't enable the guarding hook. Example:
D68911 <https://reviews.llvm.org/D68911>
...but any target has the ability to change that as needed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63382/new/

https://reviews.llvm.org/D63382





More information about the llvm-commits mailing list