--- Comment #21 from jakub at gcc dot gnu dot org 2007-11-29 21:58 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #20 from jakub at gcc dot gnu dot org 2007-11-29 21:57 ---
Subject: Bug 33434
Author: jakub
Date: Thu Nov 29 21:57:38 2007
New Revision: 130521
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=130521
Log:
PR tree-optimization/33434
* tree-inline.c (setu
--- Comment #19 from jakub at gcc dot gnu dot org 2007-11-29 18:39 ---
Have a modified patch which cures this, testing...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33434
--- Comment #18 from pinskia at gcc dot gnu dot org 2007-11-23 05:37
---
(In reply to comment #17)
> What about using the copied default SSA_NAME for the function arguments? That
> seems better and you don't need to rename the names.
Actually ignore this, I have not read the inliner c
--- Comment #17 from pinskia at gcc dot gnu dot org 2007-11-23 05:33
---
What about using the copied default SSA_NAME for the function arguments? That
seems better and you don't need to rename the names.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33434
--- Comment #16 from jakub at gcc dot gnu dot org 2007-11-16 20:29 ---
Tried to bootstrap that, and while just for make check it basically just
needed gcc_assert (!value || !TREE_SIDE_EFFECTS (value));
because e.g. some Fortran testcases have value == NULL,
bootstrap fails, e.g. on i386.
--- Comment #15 from hubicka at gcc dot gnu dot org 2007-11-08 12:42
---
The problem here is that code assumes that variable without DEFAULT_DEF is not
in SSA. Arguments that are unused also do have DEFAULT_DEF=NULL, this patch
should fix it. I am testing it now.
Honza
Index: tree-inl
--- Comment #14 from razya at il dot ibm dot com 2007-10-30 09:14 ---
Subject: Re: [4.3 Regression] inlining miscompilation
"rguenther at suse dot de" <[EMAIL PROTECTED]> wrote on 29/10/2007
15:12:36:
>
>
> --- Comment #13 from rguenther at suse dot de 2007-10-29 13:12
--
--- Comment #13 from rguenther at suse dot de 2007-10-29 13:12 ---
Subject: Re: [4.3 Regression] inlining
miscompilation
On Mon, 29 Oct 2007, razya at il dot ibm dot com wrote:
> > --- Comment #11 from rguenther at suse dot de 2007-10-29 12:14
> ---
> > Subject: Re: [4.3 R
--- Comment #12 from razya at il dot ibm dot com 2007-10-29 13:00 ---
Subject: Re: [4.3 Regression] inlining miscompilation
"rguenther at suse dot de" <[EMAIL PROTECTED]> wrote on 29/10/2007
14:14:45:
>
>
> --- Comment #11 from rguenther at suse dot de 2007-10-29 12:14
--
--- Comment #11 from rguenther at suse dot de 2007-10-29 12:14 ---
Subject: Re: [4.3 Regression] inlining
miscompilation
On Mon, 29 Oct 2007, razya at il dot ibm dot com wrote:
> --- Comment #10 from razya at il dot ibm dot com 2007-10-29 12:08 ---
> (In reply to comment #6)
--- Comment #10 from razya at il dot ibm dot com 2007-10-29 12:08 ---
(In reply to comment #6)
> Hmm, I have a question about IPA CP, should it call cfgcleanup also? It does
> not fix the problem here but it seems like a good idea. I can test a patch
> which adds the cfgcleanup if it i
--- Comment #9 from hubicka at gcc dot gnu dot org 2007-10-28 19:17 ---
Mine
--
hubicka at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned
--- Comment #8 from rguenth at gcc dot gnu dot org 2007-10-28 18:57 ---
mark_sym_for_renaming really needs some checking code for this. Or
mark_sym_for_renaming needs to properly go out-of-ssa for the symbol (in the
case of inlining separately so for the caller and the callee).
--
r
14 matches
Mail list logo
