* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
@ 2007-07-13 0:25 ` fxcoudert at gcc dot gnu dot org
2007-07-13 0:30 ` fxcoudert at gcc dot gnu dot org
` (25 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2007-07-13 0:25 UTC (permalink / raw)
To: gcc-bugs
--
fxcoudert at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Keywords| |wrong-code
Last reconfirmed|0000-00-00 00:00:00 |2007-07-13 00:25:37
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
2007-07-13 0:25 ` [Bug target/32749] " fxcoudert at gcc dot gnu dot org
@ 2007-07-13 0:30 ` fxcoudert at gcc dot gnu dot org
2007-07-13 14:50 ` hjl at lucon dot org
` (24 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2007-07-13 0:30 UTC (permalink / raw)
To: gcc-bugs
------- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-07-13 00:30 -------
There has not been any patch to the Fortran front-end during that time, making
it plausible that it's a middle-end or target bug.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
2007-07-13 0:25 ` [Bug target/32749] " fxcoudert at gcc dot gnu dot org
2007-07-13 0:30 ` fxcoudert at gcc dot gnu dot org
@ 2007-07-13 14:50 ` hjl at lucon dot org
2007-07-13 18:52 ` hjl at lucon dot org
` (23 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-13 14:50 UTC (permalink / raw)
To: gcc-bugs
------- Comment #2 from hjl at lucon dot org 2007-07-13 14:50 -------
revision 126030 works.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (2 preceding siblings ...)
2007-07-13 14:50 ` hjl at lucon dot org
@ 2007-07-13 18:52 ` hjl at lucon dot org
2007-07-13 20:02 ` hjl at lucon dot org
` (22 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-13 18:52 UTC (permalink / raw)
To: gcc-bugs
------- Comment #3 from hjl at lucon dot org 2007-07-13 18:52 -------
revision 126045 works.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (3 preceding siblings ...)
2007-07-13 18:52 ` hjl at lucon dot org
@ 2007-07-13 20:02 ` hjl at lucon dot org
2007-07-13 20:56 ` hjl at lucon dot org
` (21 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-13 20:02 UTC (permalink / raw)
To: gcc-bugs
------- Comment #4 from hjl at lucon dot org 2007-07-13 20:02 -------
revision 126056 is bad.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (4 preceding siblings ...)
2007-07-13 20:02 ` hjl at lucon dot org
@ 2007-07-13 20:56 ` hjl at lucon dot org
2007-07-13 21:53 ` hjl at lucon dot org
` (20 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-13 20:56 UTC (permalink / raw)
To: gcc-bugs
------- Comment #5 from hjl at lucon dot org 2007-07-13 20:56 -------
revision 126050 works.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (5 preceding siblings ...)
2007-07-13 20:56 ` hjl at lucon dot org
@ 2007-07-13 21:53 ` hjl at lucon dot org
2007-07-13 21:58 ` hjl at lucon dot org
` (19 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-13 21:53 UTC (permalink / raw)
To: gcc-bugs
------- Comment #6 from hjl at lucon dot org 2007-07-13 21:53 -------
revision 126054 is bad.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (6 preceding siblings ...)
2007-07-13 21:53 ` hjl at lucon dot org
@ 2007-07-13 21:58 ` hjl at lucon dot org
2007-07-13 22:07 ` rguenth at gcc dot gnu dot org
` (18 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-13 21:58 UTC (permalink / raw)
To: gcc-bugs
------- Comment #7 from hjl at lucon dot org 2007-07-13 21:57 -------
This patch
http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01977.html
is the cause. Richard, can you look into it? Thanks.
--
hjl at lucon dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |rguenther at suse dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug target/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (7 preceding siblings ...)
2007-07-13 21:58 ` hjl at lucon dot org
@ 2007-07-13 22:07 ` rguenth at gcc dot gnu dot org
2007-07-15 21:07 ` [Bug middle-end/32749] " pinskia at gcc dot gnu dot org
` (17 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2007-07-13 22:07 UTC (permalink / raw)
To: gcc-bugs
------- Comment #8 from rguenth at gcc dot gnu dot org 2007-07-13 22:07 -------
Sure, though I doubt this patch changed anything. I won't get to it until
after
the summit though.
--
rguenth at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC|rguenther at suse dot de |rguenth at gcc dot gnu dot
| |org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (8 preceding siblings ...)
2007-07-13 22:07 ` rguenth at gcc dot gnu dot org
@ 2007-07-15 21:07 ` pinskia at gcc dot gnu dot org
2007-07-16 12:28 ` rguenth at gcc dot gnu dot org
` (16 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2007-07-15 21:07 UTC (permalink / raw)
To: gcc-bugs
--
pinskia at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Component|target |middle-end
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (9 preceding siblings ...)
2007-07-15 21:07 ` [Bug middle-end/32749] " pinskia at gcc dot gnu dot org
@ 2007-07-16 12:28 ` rguenth at gcc dot gnu dot org
2007-07-16 13:25 ` hjl at lucon dot org
` (15 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2007-07-16 12:28 UTC (permalink / raw)
To: gcc-bugs
------- Comment #9 from rguenth at gcc dot gnu dot org 2007-07-16 12:27 -------
I bet that if you back up until before
Author: rguenth
Date: Sat Jun 23 18:17:57 2007
New Revision: 125974
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=125974
Log:
2007-06-23 Richard Guenther <rguenther@suse.de>
PR tree-optimization/16876
PR middle-end/29478
* tree.h (CALL_CANNOT_INLINE_P): New macro to access static_flag
for CALL_EXPRs.
the testcase will fail as well. The effect of the patch you pointed at is
just to make inlining possible again.
I'll defer this for someone else to check what goes wrong with this testcase.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (10 preceding siblings ...)
2007-07-16 12:28 ` rguenth at gcc dot gnu dot org
@ 2007-07-16 13:25 ` hjl at lucon dot org
2007-07-16 13:34 ` rguenth at gcc dot gnu dot org
` (14 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-16 13:25 UTC (permalink / raw)
To: gcc-bugs
------- Comment #10 from hjl at lucon dot org 2007-07-16 13:25 -------
Revision 125888 works:
http://gcc.gnu.org/ml/gcc-testresults/2007-06/msg00941.html
Revision 125925 is bad:
http://gcc.gnu.org/ml/gcc-testresults/2007-06/msg01013.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (11 preceding siblings ...)
2007-07-16 13:25 ` hjl at lucon dot org
@ 2007-07-16 13:34 ` rguenth at gcc dot gnu dot org
2007-07-16 17:10 ` hjl at lucon dot org
` (13 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2007-07-16 13:34 UTC (permalink / raw)
To: gcc-bugs
------- Comment #11 from rguenth at gcc dot gnu dot org 2007-07-16 13:33 -------
That makes me comfortable ;)
--
rguenth at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC|rguenth at gcc dot gnu dot |
|org |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (12 preceding siblings ...)
2007-07-16 13:34 ` rguenth at gcc dot gnu dot org
@ 2007-07-16 17:10 ` hjl at lucon dot org
2007-07-16 18:14 ` hjl at lucon dot org
` (12 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-16 17:10 UTC (permalink / raw)
To: gcc-bugs
------- Comment #12 from hjl at lucon dot org 2007-07-16 17:10 -------
revision 125905 works.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (13 preceding siblings ...)
2007-07-16 17:10 ` hjl at lucon dot org
@ 2007-07-16 18:14 ` hjl at lucon dot org
2007-07-16 18:47 ` ubizjak at gmail dot com
` (11 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-16 18:14 UTC (permalink / raw)
To: gcc-bugs
------- Comment #13 from hjl at lucon dot org 2007-07-16 18:14 -------
revision 125920 works.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (14 preceding siblings ...)
2007-07-16 18:14 ` hjl at lucon dot org
@ 2007-07-16 18:47 ` ubizjak at gmail dot com
2007-07-16 19:12 ` hjl at lucon dot org
` (10 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: ubizjak at gmail dot com @ 2007-07-16 18:47 UTC (permalink / raw)
To: gcc-bugs
------- Comment #14 from ubizjak at gmail dot com 2007-07-16 18:47 -------
(In reply to comment #13)
> revision 125920 works.
> Revision 125925 is bad:
17:06 r125925 - in /trunk/gcc: ChangeLog testsuite/gc... spop
16:25 r125924 - in /trunk/gcc: ChangeLog df-problems.c zadeck
15:47 r125923 - in /branches/gcc-4_2-branch/gcc: Chan... hjl
14:54 r125922 - in /trunk/gcc: ChangeLog testsuite/Ch... rguenth
13:18 r125921 - in /branches/gcc-4_2-branch/gcc/tests... hjl
12:31 r125920 - in /trunk/gcc: ChangeLog config/i386/... hjl
Could we place our bets now? ;)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (15 preceding siblings ...)
2007-07-16 18:47 ` ubizjak at gmail dot com
@ 2007-07-16 19:12 ` hjl at lucon dot org
2007-07-16 19:27 ` hjl at lucon dot org
` (9 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-16 19:12 UTC (permalink / raw)
To: gcc-bugs
------- Comment #15 from hjl at lucon dot org 2007-07-16 19:11 -------
revision 125924 is bad. I am testing revision 125923 now.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (16 preceding siblings ...)
2007-07-16 19:12 ` hjl at lucon dot org
@ 2007-07-16 19:27 ` hjl at lucon dot org
2007-07-16 23:26 ` zadeck at naturalbridge dot com
` (8 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: hjl at lucon dot org @ 2007-07-16 19:27 UTC (permalink / raw)
To: gcc-bugs
------- Comment #16 from hjl at lucon dot org 2007-07-16 19:27 -------
revision 125923 works. Kenny, it looks like your patch
http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01557.html
causes this regression. Can you look into it? Thanks.
--
hjl at lucon dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |zadeck at naturalbridge dot
| |com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (17 preceding siblings ...)
2007-07-16 19:27 ` hjl at lucon dot org
@ 2007-07-16 23:26 ` zadeck at naturalbridge dot com
2007-07-25 18:41 ` zadeck at naturalbridge dot com
` (7 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at naturalbridge dot com @ 2007-07-16 23:26 UTC (permalink / raw)
To: gcc-bugs
------- Comment #17 from zadeck at naturalbridge dot com 2007-07-16 23:26 -------
Subject: Re: [4.3 regression]: gfortran.dg/auto_array_1.f90
hjl at lucon dot org wrote:
> ------- Comment #16 from hjl at lucon dot org 2007-07-16 19:27 -------
> revision 125923 works. Kenny, it looks like your patch
>
> http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01557.html
>
> causes this regression. Can you look into it? Thanks.
>
>
>
I will look into this as soon as the bootstrap starts working again on
the ia-64.
Kenny
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (18 preceding siblings ...)
2007-07-16 23:26 ` zadeck at naturalbridge dot com
@ 2007-07-25 18:41 ` zadeck at naturalbridge dot com
2007-07-26 11:51 ` zadeck at naturalbridge dot com
` (6 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at naturalbridge dot com @ 2007-07-25 18:41 UTC (permalink / raw)
To: gcc-bugs
------- Comment #18 from zadeck at naturalbridge dot com 2007-07-25 18:41 -------
i am testing a patch.
--
zadeck at naturalbridge dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|unassigned at gcc dot gnu |zadeck at naturalbridge dot
|dot org |com
Status|NEW |ASSIGNED
Last reconfirmed|2007-07-13 00:25:37 |2007-07-25 18:41:41
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (19 preceding siblings ...)
2007-07-25 18:41 ` zadeck at naturalbridge dot com
@ 2007-07-26 11:51 ` zadeck at naturalbridge dot com
2007-07-26 17:28 ` seongbae dot park at gmail dot com
` (5 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at naturalbridge dot com @ 2007-07-26 11:51 UTC (permalink / raw)
To: gcc-bugs
------- Comment #19 from zadeck at naturalbridge dot com 2007-07-26 11:51 -------
Subject: Re: [4.3 regression]: gfortran.dg/auto_array_1.f90
This patch extends the fix in
http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01557.html
to handle the case of clobbers inside conditional calls.
This problem caused the regression of gfortran.dg/matmul_3.f90 on the
ia-64 in addition to the regression cited in this pr.
Tested on ppc-32, ia-64 and x86-64.
2007-07-26 Kenneth Zadeck <zadeck@naturalbridge.com>
PR middle-end/32749
* df-problems.c (df_note_bb_compute): Handle case of clobber
inside conditional call.
ok to commit?
kenny
Index: df-problems.c
===================================================================
--- df-problems.c (revision 126918)
+++ df-problems.c (working copy)
@@ -3989,7 +3989,7 @@ df_note_bb_compute (unsigned int bb_inde
/* However a may or must clobber still needs to kill the
reg so that REG_DEAD notes are later placed
appropriately. */
- else
+ else if (!(DF_REF_FLAGS (def) & (DF_REF_PARTIAL |
DF_REF_CONDITIONAL)))
bitmap_clear_bit (live, DF_REF_REGNO (def));
}
}
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (20 preceding siblings ...)
2007-07-26 11:51 ` zadeck at naturalbridge dot com
@ 2007-07-26 17:28 ` seongbae dot park at gmail dot com
2007-07-26 17:36 ` zadeck at naturalbridge dot com
` (4 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: seongbae dot park at gmail dot com @ 2007-07-26 17:28 UTC (permalink / raw)
To: gcc-bugs
------- Comment #20 from seongbae dot park at gmail dot com 2007-07-26 17:27 -------
Subject: Re: [4.3 regression]: gfortran.dg/auto_array_1.f90
On 7/26/07, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
> This patch extends the fix in
> http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01557.html
> to handle the case of clobbers inside conditional calls.
>
> This problem caused the regression of gfortran.dg/matmul_3.f90 on the
> ia-64 in addition to the regression cited in this pr.
>
> Tested on ppc-32, ia-64 and x86-64.
>
> 2007-07-26 Kenneth Zadeck <zadeck@naturalbridge.com>
>
> PR middle-end/32749
>
> * df-problems.c (df_note_bb_compute): Handle case of clobber
> inside conditional call.
>
> ok to commit?
This change is OK.
Though I wonder if we need to do similar checking
for the regular insn case below.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (21 preceding siblings ...)
2007-07-26 17:28 ` seongbae dot park at gmail dot com
@ 2007-07-26 17:36 ` zadeck at naturalbridge dot com
2007-07-26 17:56 ` seongbae dot park at gmail dot com
` (3 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at naturalbridge dot com @ 2007-07-26 17:36 UTC (permalink / raw)
To: gcc-bugs
------- Comment #21 from zadeck at naturalbridge dot com 2007-07-26 17:35 -------
Subject: Re: [4.3 regression]: gfortran.dg/auto_array_1.f90
Seongbae Park (???, ???) wrote:
> On 7/26/07, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
>> This patch extends the fix in
>> http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01557.html
>> to handle the case of clobbers inside conditional calls.
>>
>> This problem caused the regression of gfortran.dg/matmul_3.f90 on the
>> ia-64 in addition to the regression cited in this pr.
>>
>> Tested on ppc-32, ia-64 and x86-64.
>>
>> 2007-07-26 Kenneth Zadeck <zadeck@naturalbridge.com>
>>
>> PR middle-end/32749
>>
>> * df-problems.c (df_note_bb_compute): Handle case of clobber
>> inside conditional call.
>>
>> ok to commit?
>
> This change is OK.
> Though I wonder if we need to do similar checking
> for the regular insn case below.
No the checking is done in df_create_unused_note. The only reason you
have to do it here is that you are not calling that.
thanks
kenny
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (22 preceding siblings ...)
2007-07-26 17:36 ` zadeck at naturalbridge dot com
@ 2007-07-26 17:56 ` seongbae dot park at gmail dot com
2007-07-27 17:22 ` zadeck at gcc dot gnu dot org
` (2 subsequent siblings)
26 siblings, 0 replies; 28+ messages in thread
From: seongbae dot park at gmail dot com @ 2007-07-26 17:56 UTC (permalink / raw)
To: gcc-bugs
------- Comment #22 from seongbae dot park at gmail dot com 2007-07-26 17:56 -------
Subject: Re: [4.3 regression]: gfortran.dg/auto_array_1.f90
On 7/26/07, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
> Seongbae Park (???, ???) wrote:
> > On 7/26/07, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
> >> This patch extends the fix in
> >> http://gcc.gnu.org/ml/gcc-patches/2007-06/msg01557.html
> >> to handle the case of clobbers inside conditional calls.
> >>
> >> This problem caused the regression of gfortran.dg/matmul_3.f90 on the
> >> ia-64 in addition to the regression cited in this pr.
> >>
> >> Tested on ppc-32, ia-64 and x86-64.
> >>
> >> 2007-07-26 Kenneth Zadeck <zadeck@naturalbridge.com>
> >>
> >> PR middle-end/32749
> >>
> >> * df-problems.c (df_note_bb_compute): Handle case of clobber
> >> inside conditional call.
> >>
> >> ok to commit?
> >
> > This change is OK.
> > Though I wonder if we need to do similar checking
> > for the regular insn case below.
> No the checking is done in df_create_unused_note. The only reason you
> have to do it here is that you are not calling that.
>
> thanks
Now that I look at df_create_unused_note,
this patch smells a bit - because the condition
inside the for loop looks identical to df_create_unused_node.
I think it would be cleaner if we split the live vector update
into a separate function. i.e. attached patch (untested).
------- Comment #23 from seongbae dot park at gmail dot com 2007-07-26 17:56 -------
Created an attachment (id=13986)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13986&action=view)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (23 preceding siblings ...)
2007-07-26 17:56 ` seongbae dot park at gmail dot com
@ 2007-07-27 17:22 ` zadeck at gcc dot gnu dot org
2007-07-27 17:30 ` zadeck at naturalbridge dot com
2007-07-27 17:34 ` zadeck at naturalbridge dot com
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at gcc dot gnu dot org @ 2007-07-27 17:22 UTC (permalink / raw)
To: gcc-bugs
------- Comment #24 from zadeck at gcc dot gnu dot org 2007-07-27 17:22 -------
Subject: Bug 32749
Author: zadeck
Date: Fri Jul 27 17:22:14 2007
New Revision: 126987
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=126987
Log:
2007-07-26 Kenneth Zadeck <zadeck@naturalbridge.com>
PR middle-end/32749
* df-problems.c (df_create_unused_note): Removed do_not_gen parm
and the updating of the live and do_not_gen sets.
(df_note_bb_compute): Added updating of live and do_not_gen sets
for regular defs so that the case of clobber inside conditional
call is processed correctly.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/df-problems.c
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (24 preceding siblings ...)
2007-07-27 17:22 ` zadeck at gcc dot gnu dot org
@ 2007-07-27 17:30 ` zadeck at naturalbridge dot com
2007-07-27 17:34 ` zadeck at naturalbridge dot com
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at naturalbridge dot com @ 2007-07-27 17:30 UTC (permalink / raw)
To: gcc-bugs
------- Comment #25 from zadeck at naturalbridge dot com 2007-07-27 17:29 -------
Subject: Re: [4.3 regression]: gfortran.dg/auto_array_1.f90
This patch rearranges the updating of the local dataflow info when
building reg_dead notes. The need for this was that processing was not
correctly handled for clobbers that occurred within conditional call
insns. A rare case but one that at least happens on the ia-64.
This patch not only fixes the regressions listed in pr32749, but also
fixes the gfortran.dg/matmul_3.f90 on the ia-64 regressions.
This patch was bootstrapped and regression tested yesterday on x86-64
and ia-64 and was again bootstrapped this morning on x86-64 (just to
make sure there were no interactions with richard sandiford's fixes to
closely related code that was just committed.)
Committed as revision 126987.
Kenny
2007-07-26 Kenneth Zadeck <zadeck@naturalbridge.com>
PR middle-end/32749
* df-problems.c (df_create_unused_note): Removed do_not_gen parm
and the updating of the live and do_not_gen sets.
(df_note_bb_compute): Added updating of live and do_not_gen sets
for regular defs so that the case of clobber inside conditional
call is processed correctly.
Index: df-problems.c
===================================================================
--- df-problems.c (revision 126979)
+++ df-problems.c (working copy)
@@ -3868,13 +3868,12 @@ df_set_dead_notes_for_mw (rtx insn, rtx
}
-/* Create a REG_UNUSED note if necessary for DEF in INSN updating LIVE
- and DO_NOT_GEN. Do not generate notes for registers in artificial
- uses. */
+/* Create a REG_UNUSED note if necessary for DEF in INSN updating
+ LIVE. Do not generate notes for registers in ARTIFICIAL_USES. */
static rtx
df_create_unused_note (rtx insn, rtx old, struct df_ref *def,
- bitmap live, bitmap do_not_gen, bitmap artificial_uses)
+ bitmap live, bitmap artificial_uses)
{
unsigned int dregno = DF_REF_REGNO (def);
@@ -3899,12 +3898,6 @@ df_create_unused_note (rtx insn, rtx old
#endif
}
- if (!(DF_REF_FLAGS (def) & (DF_REF_MUST_CLOBBER + DF_REF_MAY_CLOBBER)))
- bitmap_set_bit (do_not_gen, dregno);
-
- /* Kill this register if it is not a subreg store or conditional store. */
- if (!(DF_REF_FLAGS (def) & (DF_REF_PARTIAL | DF_REF_CONDITIONAL)))
- bitmap_clear_bit (live, dregno);
return old;
}
@@ -3915,7 +3908,7 @@ df_create_unused_note (rtx insn, rtx old
static void
df_note_bb_compute (unsigned int bb_index,
- bitmap live, bitmap do_not_gen, bitmap artificial_uses)
+ bitmap live, bitmap do_not_gen, bitmap artificial_uses)
{
basic_block bb = BASIC_BLOCK (bb_index);
rtx insn;
@@ -4012,17 +4005,17 @@ df_note_bb_compute (unsigned int bb_inde
for (def_rec = DF_INSN_UID_DEFS (uid); *def_rec; def_rec++)
{
struct df_ref *def = *def_rec;
- if (!(DF_REF_FLAGS (def) & (DF_REF_MUST_CLOBBER |
DF_REF_MAY_CLOBBER)))
- old_unused_notes
- = df_create_unused_note (insn, old_unused_notes,
- def, live, do_not_gen,
- artificial_uses);
-
- /* However a may or must clobber still needs to kill the
- reg so that REG_DEAD notes are later placed
- appropriately. */
- else
- bitmap_clear_bit (live, DF_REF_REGNO (def));
+ unsigned int dregno = DF_REF_REGNO (def);
+ if (!DF_REF_FLAGS_IS_SET (def, DF_REF_MUST_CLOBBER |
DF_REF_MAY_CLOBBER))
+ {
+ old_unused_notes
+ = df_create_unused_note (insn, old_unused_notes,
+ def, live, artificial_uses);
+ bitmap_set_bit (do_not_gen, dregno);
+ }
+
+ if (!DF_REF_FLAGS_IS_SET (def, DF_REF_PARTIAL |
DF_REF_CONDITIONAL))
+ bitmap_clear_bit (live, dregno);
}
}
else
@@ -4043,10 +4036,16 @@ df_note_bb_compute (unsigned int bb_inde
for (def_rec = DF_INSN_UID_DEFS (uid); *def_rec; def_rec++)
{
struct df_ref *def = *def_rec;
+ unsigned int dregno = DF_REF_REGNO (def);
old_unused_notes
= df_create_unused_note (insn, old_unused_notes,
- def, live, do_not_gen,
- artificial_uses);
+ def, live, artificial_uses);
+
+ if (!DF_REF_FLAGS_IS_SET (def, DF_REF_MUST_CLOBBER |
DF_REF_MAY_CLOBBER))
+ bitmap_set_bit (do_not_gen, dregno);
+
+ if (!DF_REF_FLAGS_IS_SET (def, DF_REF_PARTIAL |
DF_REF_CONDITIONAL))
+ bitmap_clear_bit (live, dregno);
}
}
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Bug middle-end/32749] [4.3 regression]: gfortran.dg/auto_array_1.f90
2007-07-12 23:50 [Bug target/32749] New: [4.3 regression]: gfortran.dg/auto_array_1.f90 hjl at lucon dot org
` (25 preceding siblings ...)
2007-07-27 17:30 ` zadeck at naturalbridge dot com
@ 2007-07-27 17:34 ` zadeck at naturalbridge dot com
26 siblings, 0 replies; 28+ messages in thread
From: zadeck at naturalbridge dot com @ 2007-07-27 17:34 UTC (permalink / raw)
To: gcc-bugs
------- Comment #26 from zadeck at naturalbridge dot com 2007-07-27 17:33 -------
revision 126987
--
zadeck at naturalbridge dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32749
^ permalink raw reply [flat|nested] 28+ messages in thread