requested changes to this pull request.

If I understood correctly, what you're trying to do here is to apply an icmp 
fold early before the sext -> zext information gets lost. I don't think this is 
the correct way to approach the problem. The correct way is to preserve the 
fact that the operand is non-negative when converting to zext, which would 
allow the InstCombine fold to reliably undo this. In fact, there is a pending 
patch for this at
cfe-commits mailing list

Reply via email to