--- Additional Comments From matz at suse dot de 2005-04-19 16:51 ---
I agree with most of what Jim said. Except for the part that we maybe don't
have to fix the reload issue, when we fix usage of the uninitialized register
for piecewise struct initialization. The latter will fix this
--- Additional Comments From wilson at gcc dot gnu dot org 2005-04-19
06:22 ---
The reload patch seems reasonable, though not very satisfying. It is only
disabling some optimizations within reload, and hence should be somewhat safe.
Though it is still a reload patch, and all reload pat
--- Additional Comments From matz at suse dot de 2005-04-18 14:51 ---
>From
http://gcc.gnu.org/ml/gcc-patches/2005-04/msg01508.html
where I submitted the patch:
the problem in khtml. I've bootstrapped it with gcc 4.0 on
i686,x86_64,ppc,ppc64,ia64,s390 (s390x was breaking for d
--- Additional Comments From mmitchel at gcc dot gnu dot org 2005-04-18
14:42 ---
Michael, have you run the GCC testsuite with your patch? If not, please run on
several platforms and confirm that you get no regressions.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20973
--- Additional Comments From matz at suse dot de 2005-04-18 14:22 ---
This patch fixes the regressions in khtml for us.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20973
--- Additional Comments From mueller at kde dot org 2005-04-13 01:54
---
will test the patch with a KDE recompilation.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20973
--- Additional Comments From matz at suse dot de 2005-04-12 19:47 ---
I have a patch for reload, which fixes the bug, when looking at the dumps.
At least now find_reg is used for the insn in question, which also evicts
pseudos using the same reg as the chosen final reg_rtx. I have only
--- Additional Comments From matz at suse dot de 2005-04-12 19:05 ---
The problem is in reload.c:find_dummy_reload. It tries to use the input reg
as reload register for an in-out reload and has certain conditions when it
can't do so:
/* Consider using IN if OUT was not acceptable
--- Additional Comments From jakub at gcc dot gnu dot org 2005-04-12 18:01
---
For those who want a quick workaround, changing khtmllayout.h to:
-Length(LengthType t) { l.value = 0; l.type = t; l.quirk = 0; }
+Length(LengthType t) { _length = 0; l.type = t; }
Len
--- Additional Comments From matz at suse dot de 2005-04-12 17:37 ---
Another mail:
I think the dust settles a bit. We have this situation in
.t69.final_cleanup (excerpt from applyRule):
struct Length D.83927;
:;
D.83927.D.21947.l.value = (int) primitiveValue->m_value.num;
--- Additional Comments From matz at suse dot de 2005-04-12 17:34 ---
Here some mails we exchanged:
Adding something complex seems to change how
things are allocated or REG_DEAD notes distributed or something like this.
If we add 'kdDebug( 6080 ) << "applying property " << id <<
--
What|Removed |Added
OtherBugsDependingO||20949
nThis||
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20973
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-04-12
17:33 ---
Reload, wow, not that unexcepted really.
--
What|Removed |Added
Keywords|
13 matches
Mail list logo