From patchwork Tue Jul 13 14:09:17 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [committed] Fix auto-inc-dec in inline-asm change
Date: Tue, 13 Jul 2010 04:09:17 -0000
From: Jakub Jelinek <jakub@redhat.com>
X-Patchwork-Id: 58779
Message-Id: <20100713140917.GZ20208@tyan-ft48-01.lab.bos.redhat.com>
To: gcc-patches@gcc.gnu.org

Hi!

The documentation and intent as well as asm_operand_ok implement either
< or > allowing side-effects, while constrain_operands mistakenly
implemented requirement that both < and > are present for that.

Fixed thusly, committed as obvious.

2010-07-13  Jakub Jelinek  <jakub@redhat.com>

	PR testsuite/44701
	* recog.c (constrain_operands): Allow side-effects in memory
	operands if either < or > constraint is used, rather than if
	both < and > is used.


	Jakub


--- gcc/recog.c.jj	2010-07-05 12:37:02.000000000 +0200
+++ gcc/recog.c	2010-07-13 13:16:07.000000000 +0200
@@ -2742,7 +2742,7 @@ constrain_operands (int strict)
 			case PRE_MODIFY:
 			case POST_MODIFY:
 			  if (strchr (recog_data.constraints[opno], '<') == NULL
-			      || strchr (recog_data.constraints[opno], '>')
+			      && strchr (recog_data.constraints[opno], '>')
 				 == NULL)
 			    return 0;
 			  break;