--- Additional Comments From matz at suse dot de 2005-06-10 11:50 ---
Thanks. Sorry, I couldn't revert as you suggested, as I became ill the day
after noticing the problem :-(
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19768
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-06-08
18:03 ---
(In reply to comment #14)
Michael could you revert it, I have no time to do so today, I have exams to
work on all day today.
I reverted it after getting approval from Mark.
So this should be fixed
--- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-06-08
18:04 ---
Subject: Bug 19768
CVSROOT:/cvs/gcc
Module name:gcc
Branch: gcc-4_0-branch
Changes by: [EMAIL PROTECTED] 2005-06-08 18:04:19
Modified files:
gcc:
--- Additional Comments From matz at suse dot de 2005-06-06 06:18 ---
This happens again. I've seen it in a 20050603 4.0 compiler (compiled
with --enable-checking). It was not happening with the 20040512 compiler
from the 4.0 branch.
--
--- Additional Comments From matz at suse dot de 2005-06-06 06:19 ---
20050512 was the working one, I meant.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19768
--- Additional Comments From matz at suse dot de 2005-06-06 06:35 ---
I know. Andrew: when you backported this patch:
PR tree-optimization/21085
* fold-const (fold): Don't change X % -C to X % C if C has
overflowed.
you accidentally also checked in a change to
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-06-06
10:51 ---
(In reply to comment #13)
I know. Andrew: when you backported this patch:
PR tree-optimization/21085
* fold-const (fold): Don't change X % -C to X % C if C has
overflowed.
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-03
15:24 ---
Fixed. Thanks for your report.
--
What|Removed |Added
Status|ASSIGNED
--- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-02-03
15:26 ---
Subject: Bug 19768
CVSROOT:/cvs/gcc
Module name:gcc
Changes by: [EMAIL PROTECTED] 2005-02-03 15:26:30
Modified files:
gcc: tree-ssa-dse.c ChangeLog
--
What|Removed |Added
CC||pinskia at gcc dot gnu dot
||org
Component|c++
--- Additional Comments From tsarkov at cs dot man dot ac dot uk
2005-02-02 19:29 ---
Created an attachment (id=8128)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=8128action=view)
Proprocessed source file
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19768
--
What|Removed |Added
Keywords||EH
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19768
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
19:42 ---
Confirmed, DSE is messing up somehow and not copying the bit once removing a
store.
--
What|Removed |Added
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
20:13 ---
Here is the most reduced testcase that I could come up with:
struct LeveLogger
{
int currentLevel;
};
extern LeveLogger LL;
struct gg
{
~gg ( void )
{ LL.currentLevel = 1; }
};
void f(void);
void g
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
20:22 ---
Here is a simple patch to fix up the error in DSE:
Index: tree-ssa-dse.c
===
RCS file: /cvs/gcc/gcc/gcc/tree-ssa-dse.c,v
retrieving revision
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
20:41 ---
My fix was not always correct. Here is a fix which is more correct and works
for this testcase also:
Index: tree-ssa-dse.c
===
RCS file:
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
21:21 ---
(In reply to comment #5)
My fix was not always correct. Here is a fix which is more correct and
works for this testcase also:
Actually this can clear the PHI in abnormal phi flag when we shoud not.
I
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
21:27 ---
Correct patch here: http://gcc.gnu.org/ml/gcc-patches/2005-02/msg00186.html
which makes
sure that we don't change SSA_NAME_OCCURES_IN_ABNORMAL_PHI from true to false.
--
What|Removed
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-02-02
23:58 ---
Mine.
--
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |pinskia at gcc
19 matches
Mail list logo