public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "abel at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug rtl-optimization/52203] ICE: in reset_sched_cycles_in_current_ebb, at sel-sched.c:7136 with -fsel-sched-pipelining -fselective-scheduling2 and other custom flags
Date: Tue, 14 Feb 2012 06:57:00 -0000	[thread overview]
Message-ID: <bug-52203-4-6RoGgwHqX7@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-52203-4@http.gcc.gnu.org/bugzilla/>

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52203

--- Comment #8 from Andrey Belevantsev <abel at gcc dot gnu.org> 2012-02-14 06:56:10 UTC ---
Sorry, I didn't explain clearly from the start.  Regarding the backend, I just
wanted to double check that for the given insn not having a reservation is
correct.  Now that Uros confirmed this, I can proceed with the scheduler patch.

Uros and Steven, regarding the big picture, you can read the thread on PR
49014, starting from http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01801.html
(our first discussions with Bernd) and initial patch with the remainder of the
thread at http://gcc.gnu.org/ml/gcc-patches/2011-07/msg00045.html.  Basically,
we allow insns that recog as >=0 but do not change the DFA state, such insns do
not get counted against issue rate.  I was wondering whether we could mark such
insns specially in the MD files and then assert that any other real insn should
affect the DFA.  The reason for this was that at least three real backend bugs
(missing DFA attributes) were fixed due to the more strict checking the
selective scheduler makes wrt the Haifa scheduler.  

Suprisingly, I could bootstrap x86-64 with just some hours of work with the
patch from http://gcc.gnu.org/ml/gcc-patches/2011-07/msg00045.html (the patch
marks the offending insns with the new attribute and adds an assert; Uros, if
you can spot any insns there that indeed should have a reservation, I can make
a patch fixing those).  However, at the time I gave in pursuing this work
further, as nobody was very fascinated with the idea and I didn't have much
free time.


  parent reply	other threads:[~2012-02-14  6:57 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-10 22:19 [Bug rtl-optimization/52203] New: " zsojka at seznam dot cz
2012-02-10 22:43 ` [Bug rtl-optimization/52203] " steven at gcc dot gnu.org
2012-02-11 12:53 ` abel at gcc dot gnu.org
2012-02-13  8:49 ` abel at gcc dot gnu.org
2012-02-13  8:49 ` abel at gcc dot gnu.org
2012-02-13 15:54 ` ubizjak at gmail dot com
2012-02-13 16:01 ` ubizjak at gmail dot com
2012-02-13 16:02 ` ubizjak at gmail dot com
2012-02-13 21:14 ` steven at gcc dot gnu.org
2012-02-14  6:57 ` abel at gcc dot gnu.org [this message]
2012-03-07 12:01 ` abel at gcc dot gnu.org
2012-03-07 12:03 ` abel at gcc dot gnu.org
2012-04-13  9:37 ` abel at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-52203-4-6RoGgwHqX7@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).