[PATCH] D32098: [InstCombine] Use less bitwise operations to handle Instruction::SExt in SimplifyDemandedUseBits. Other improvements.

[Davide Italiano via Phabricator via llvm-commits]

davide requested changes to this revision.
davide added a comment.
This revision now requires changes to proceed.

To be honest, I'm not entirely sure this cleanup is a net win.  The fact we have to duplicate the zext logic is a little annoying. 
I don't feel strongly about rejecting this patch, but I don't feel strongly about accepting either. I guess David as instcombine owner has the final word on this one.


https://reviews.llvm.org/D32098