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.
next prev 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: linkBe 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).