public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
* Re: target/4860: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi
@ 2002-01-22 15:07 pb
0 siblings, 0 replies; 3+ messages in thread
From: pb @ 2002-01-22 15:07 UTC (permalink / raw)
To: gcc-bugs, gcc-prs, kediger, nobody, pb
Old Synopsis: Internal Compiler Error with ARM g++ cross compiler
New Synopsis: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi
Responsible-Changed-From-To: unassigned->pb
Responsible-Changed-By: pb
Responsible-Changed-When: Tue Jan 22 15:07:15 2002
Responsible-Changed-Why:
.
State-Changed-From-To: open->analyzed
State-Changed-By: pb
State-Changed-When: Tue Jan 22 15:07:15 2002
State-Changed-Why:
Still occurs with 3.0 branch.
This loop in sjlj_mark_call_sites (about line 2135) is too simple minded:
/* Search backward for the first set of a register in this set. */
while (nparm_regs)
{
before = PREV_INSN (before);
/* Given that we've done no other optimizations yet,
the arguments should be immediately available. */
if (GET_CODE (before) == CODE_LABEL)
abort ();
p = single_set (before);
if (p && GET_CODE (SET_DEST (p)) == REG
&& REGNO (SET_DEST (p)) < FIRST_PSEUDO_REGISTER
&& TEST_HARD_REG_BIT (parm_regs, REGNO (SET_DEST (p))))
{
CLEAR_HARD_REG_BIT (parm_regs, REGNO (SET_DEST (p)));
nparm_regs--;
}
}
What happens in this case is that several of the SETs for argument registers have been wrapped up in a PARALLEL in order to make them come out as an LDM, which stops single_set from seeing them.
This code has already been restructured on the trunk to use note_stores, which avoids the problem. See:
Sun Jul 29 23:26:50 CEST 2001 Jan Hubicka <jh@suse.cz>
* rtlanal.c (parms_set, find_first_parameter_load): Break out from...;
handle multiple sets.
* except.c (sjlj_mark_call_sites): .... here.
* gcse.c (insert_insn_end_bb): Use find_first_parameter_load.
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=4860
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: target/4860: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi
@ 2002-02-13 14:28 pb
0 siblings, 0 replies; 3+ messages in thread
From: pb @ 2002-02-13 14:28 UTC (permalink / raw)
To: gcc-bugs, gcc-prs, kediger, pb
Synopsis: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi
State-Changed-From-To: feedback->closed
State-Changed-By: pb
State-Changed-When: Wed Feb 13 14:28:57 2002
State-Changed-Why:
Fixed in 3.0 branch
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=4860
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: target/4860: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi
@ 2002-02-11 5:51 pb
0 siblings, 0 replies; 3+ messages in thread
From: pb @ 2002-02-11 5:51 UTC (permalink / raw)
To: gcc-bugs, gcc-prs, kediger, pb
Synopsis: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi
State-Changed-From-To: analyzed->feedback
State-Changed-By: pb
State-Changed-When: Mon Feb 11 05:51:16 2002
State-Changed-Why:
proposed patch at:
http://gcc.gnu.org/ml/gcc-patches/2002-02/msg00627.html
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=4860
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2002-02-13 22:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-01-22 15:07 target/4860: [ARM; 3.0] sjlj_mark_call_sites thwarted by load-multi pb
2002-02-11 5:51 pb
2002-02-13 14:28 pb
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).