--- Comment #12 from amacleod at redhat dot com 2008-01-17 21:54 ---
Bah. I missed getting the PR in the checkin message.
Committed the proposed patch and the testcase as revision 131610. Bootstrapped
and regression tested on x86_64-linux-gnu
Follow on conversations about const funct
--- Comment #11 from amacleod at redhat dot com 2008-01-16 22:44 ---
One could also argue that this exposes two bugs.
First, the const function should not be returning true for tree_could_throw_p()
second, SCCVN and PRE don't agree on whether an expression that can throw can
be regenera
--- Comment #10 from amacleod at redhat dot com 2008-01-16 21:27 ---
Sure, if that the case, then the bug is in tree_could_throw_p() and we can fix
it there :-)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34648
--- Comment #9 from rguenther at suse dot de 2008-01-16 21:18 ---
Subject: Re: [4.3 Regression] ICE in
find_or_generate_expression
On Wed, 16 Jan 2008, amacleod at redhat dot com wrote:
> --- Comment #8 from amacleod at redhat dot com 2008-01-16 20:53 ---
> Created an attach
--- Comment #8 from amacleod at redhat dot com 2008-01-16 20:53 ---
Created an attachment (id=14951)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14951&action=view)
proposed patch for sccvn
I did some analysis, and then verified with dberlin that I was on the right
track.
The ro
--- Comment #7 from rguenth at gcc dot gnu dot org 2008-01-12 19:48 ---
Maybe sth for steven, as he was asking for bugs to look at... ;)
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from dberlin at gcc dot gnu dot org 2008-01-12 19:33 ---
Subject: Re: [4.3 Regression] ICE in find_or_generate_expression
On 12 Jan 2008 19:13:56 -, rguenth at gcc dot gnu dot org
<[EMAIL PROTECTED]> wrote:
>
>
> --- Comment #5 from rguenth at gcc dot gnu dot org
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-01-12 19:13 ---
Danny, maybe you mis-understood. We try to generate *VH.98, but VH.98 is
indeed
not in avail_out[19], but created for example as
exp_gen[4] := { x_7(D) (VH.72) , D.1661_4 (VH.98) , *VH.74 (VH.75) , a.1_6
(VH.76) ,
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-01-12 18:02 ---
Also ICEs cc1plus with plain -O2 the same way.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34648
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Priority|P3 |P1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34648
--- Comment #3 from dberlin at gcc dot gnu dot org 2008-01-03 18:16 ---
It is never okay to base an existing expression on an SSA_NAME alone, which is
why we avoid it.
If the SSA_NAME was available, it would have been in the AVAIL set and been
found by the "find" part of find_or_generat
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
CC||pinskia at gcc dot gnu dot
|
--- Comment #2 from rguenth at gcc dot gnu dot org 2008-01-03 11:05 ---
We are creating *VH.97 by pieces but fail to create the 'piece' for VH.97
because
we don't consider a SSA_NAME ok for it.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34648
--- Comment #1 from rguenth at gcc dot gnu dot org 2008-01-03 11:02 ---
Confirmed. We only have SSA_NAMEs in the expression set, which we do not PRE.
Danny, is this supposed not to happen somehow?
--
rguenth at gcc dot gnu dot org changed:
What|Removed
14 matches
Mail list logo
