public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [committed] Fix auto-inc-dec in inline-asm change
@ 2010-07-13 14:07 Jakub Jelinek
  0 siblings, 0 replies; only message in thread
From: Jakub Jelinek @ 2010-07-13 14:07 UTC (permalink / raw)
  To: gcc-patches

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.

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

	Jakub

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2010-07-13 14:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-13 14:07 [committed] Fix auto-inc-dec in inline-asm change Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).