On 02/24/2016 03:05 PM, Jakub Jelinek wrote:
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?

Yes.

Jason

Reply via email to