[Patch]New InstCombine pattern for Icmp

Chandler Carruth chandlerc at google.com
Wed Aug 13 16:29:57 PDT 2014


On Wed, Aug 13, 2014 at 4:26 PM, Gerolf Hoflehner <ghoflehner at apple.com>
wrote:

> Well, technically the LGTM is actually unlimited meant to taking effect in
> 24hrs with
> confidence. There is just a ponder period to allow that confidence to be
> shattered.
>

FWIW, this is one reason why I'm a huge fan of a revert-friendly
development process. If someone realizes why this isn't a good idea, they
can (and should!) provide post-commit review, and folks can quickly revert
to try to understand and/or address the concern that came up.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140813/96945c05/attachment.html>


More information about the llvm-commits mailing list