--- Comment #18 from hjl at gcc dot gnu dot org 2010-02-07 04:44 ---
Subject: Bug 42248
Author: hjl
Date: Sun Feb 7 04:41:22 2010
New Revision: 156562
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=156562
Log:
Backport testcases from mainline to 4.4.
2010-02-06 H.J. Lu
--- Comment #14 from rguenth at gcc dot gnu dot org 2010-01-17 14:20
---
Any news on the updated patch?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42248
--- Comment #15 from pthaugen at gcc dot gnu dot org 2010-01-17 16:54
---
Updated patch works for testcase and bootstrap/regtest on PPC passed. Thx.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42248
--- Comment #16 from rguenth at gcc dot gnu dot org 2010-01-17 17:00
---
Subject: Bug 42248
Author: rguenth
Date: Sun Jan 17 17:00:47 2010
New Revision: 155984
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=155984
Log:
2010-01-17 Richard Guenther rguent...@suse.de
PR
--- Comment #17 from rguenth at gcc dot gnu dot org 2010-01-17 17:01
---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #13 from rguenth at gcc dot gnu dot org 2010-01-15 09:41
---
Created an attachment (id=19606)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19606action=view)
fixed patch
Ah, obvious oversight - I forgot to replace the realpart parameter in the
vector of parameters.
--- Comment #9 from rguenth at gcc dot gnu dot org 2010-01-14 14:34 ---
Ugh.
args = copy_list (args);
simple-minded copies all PARM_DECLs in the function - but of course the
actual functions parameter list is not adjusted, this is only a temporary
list used for parameter setup.
So
--- Comment #10 from rguenth at gcc dot gnu dot org 2010-01-14 15:15
---
Created an attachment (id=19597)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19597action=view)
patch 2)
Implements 2) - can you test that as well? Another possibility would be to
move the split complex
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Component|tree-optimization |middle-end
Target Milestone|--- |4.5.0
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |rguenth at gcc dot gnu dot
|dot org
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Priority|P3 |P1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42248
--- Comment #12 from pthaugen at gcc dot gnu dot org 2010-01-14 17:29
---
The first patch appeared to work, resulting in correct ordering, but the second
patch had some issues. It also corrected the original ordering, but introduced
some new incorrect code for handling the args coming
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-01-13 22:44 ---
It rightfully returns false - they are two different PARM_DECLs. This is btw
a regression - marked as such.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-01-13 22:51 ---
Can you check -fdump-rtl-all-uid dumps where they start to differ? It seems
to work ok on i?86-linux.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42248
--- Comment #6 from rguenth at gcc dot gnu dot org 2010-01-13 23:22 ---
Happens in split_complex_args it seems. What this does is changing a
Complex PARM_DECL into a component PARM_DECL and add another for the remaining
component. But it doesn't adjust the function IL nor the
--- Comment #7 from rguenth at gcc dot gnu dot org 2010-01-13 23:29 ---
10 bonus points if you make ppc not use that weird feature. I can't see
a single good reason for its existence.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42248
--- Comment #8 from pinskia at gcc dot gnu dot org 2010-01-13 23:48 ---
split_complex_args came in from:
r67367 | aldyh | 2003-06-03 04:14:07 -0700 (Tue, 03 Jun 2003) | 24 lines
2003-06-03 Aldy Hernandez
17 matches
Mail list logo