Patchwork [committed] Fix auto-inc-dec in inline-asm change

login
register
mail settings
Submitter Jakub Jelinek
Date July 13, 2010, 2:09 p.m.
Message ID <20100713140917.GZ20208@tyan-ft48-01.lab.bos.redhat.com>
Download mbox | patch
Permalink /patch/58779/
State New
Headers show

Comments

Jakub Jelinek - July 13, 2010, 2:09 p.m.
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

Patch

--- 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;