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

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

Reply via email to