Georg-Johann Lay schrieb:
Jeff Law schrieb:
On 01/25/2013 11:41 AM, Georg-Johann Lay wrote:
PR54814 causes spill fails because reload.c:find_valid_class_1 tests only one
hard register instead of all hard registers of regno:mode in rclass:

http://gcc.gnu.org/PR54814


The patch was originally worked out by Bernd Schmidt and fixed a problem
introduced in

http://gcc.gnu.org/r190252

The patch is bootstrapped and tested on x86_64-linux and also fixes the spill
fails that originally occurred on avr-unknown-one.

Ok to apply?


    PR other/54814
    * reload.c (find_valid_class_1): Use in_hard_reg_set_p instead of
    TEST_HARD_REG_BIT.
Is this a regression relative to a prior version of GCC?

Yes, it's 4.8 regression. 4.7 works and r190252 was only applied to 4.8 trunk.

If not, it'll probably need release manager approval before it can go in.

Please attach your patch to PR54814 and attach PR 54814 to the 4.9 pending patches meta bug.

Does this mean the fix is rejected for 4.8?

I found no "4.9 meta-bug" in the 47 meta-bugs. You have th PR?

http://gcc.gnu.org/bugzilla/buglist.cgi?keywords=meta-bug%2C%20&keywords_type=allwords&list_id=51998&cf_known_to_fail_type=allwords&cf_known_to_work_type=allwords&query_format=advanced&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=SUSPENDED&bug_status=WAITING&bug_status=REOPENED&product=gcc

FYI, this bug breaks the avr port almost completely. Many real-world programs ICE.

Reply via email to