On Wed, Feb 24, 2016 at 02:57:49PM -0500, Jason Merrill wrote: > On 02/24/2016 02:05 PM, Jakub Jelinek wrote: > >+ && integer_zerop (tree_strip_nop_conversions (TREE_OPERAND (org_x, > >+ 1)))) > > Why check this? I think we want this handling for all TREE_NO_WARNING > comparisons.
There probably aren't any other anyway, so it is really not needed.
Is the patch ok with those two lines removed and ) moved?
Jakub
