Patchwork Fix PR57268

login
register
mail settings
Submitter Jakub Jelinek
Date June 1, 2013, 8:11 a.m.
Message ID <20130601081124.GY1493@tucnak.redhat.com>
Download mbox | patch
Permalink /patch/248066/
State New
Headers show

Comments

Jakub Jelinek - June 1, 2013, 8:11 a.m.
On Sat, Jun 01, 2013 at 08:39:58AM +0400, Dinar Temirbulatov wrote:
> oh, This is my mistake I should have bootstrap the compiler.

That is a requirement for most of commits, see
http://gcc.gnu.org/contribute.html for the exact rules.

I am investigating the problem.

I'd guess you don't want to flush on DEBUG_INSN_Ps, because then you'd flush
differently between -g and -g0.  So perhaps something like:
but I haven't tested it.  Note, in your patch the closing } was misplaced,
it is supposed to line up with the opening {.

Anyway, if you can't fix this really quickly, please revert your patch.
Bootstrap failure on the most commonly used GCC target by developers
should never be taken lightly.

	Jakub

Patch

--- gcc/sched-deps.c	2013-06-01 10:03:00.678514521 +0200
+++ gcc/sched-deps.c	2013-06-01 10:05:03.119864726 +0200
@@ -2694,10 +2694,11 @@  sched_analyze_2 (struct deps_desc *deps,
 	  {
 	    if ((deps->pending_read_list_length
 		 + deps->pending_write_list_length)
-		> MAX_PENDING_LIST_LENGTH)
+		> MAX_PENDING_LIST_LENGTH
+		&& !DEBUG_INSN_P (insn))
 	      flush_pending_lists (deps, insn, true, true);
 	    add_insn_mem_dependence (deps, true, insn, x);
-	}
+	  }
 
 	sched_analyze_2 (deps, XEXP (x, 0), insn);