[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-19 Thread matz at suse dot de
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-18 Thread wilson at gcc dot gnu dot org
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-18 Thread matz at suse dot de
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-18 Thread mmitchel at gcc dot gnu dot org
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-18 Thread matz at suse dot de
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread mueller at kde dot org
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread matz at suse dot de
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread matz at suse dot de
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread jakub at gcc dot gnu dot org
--- 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

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread matz at suse dot de
--- 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;

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread matz at suse dot de
--- 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 <<

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread pinskia at gcc dot gnu dot org
-- What|Removed |Added OtherBugsDependingO||20949 nThis|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20973

[Bug regression/20973] [4.0/4.1 Regression] kdelibs (khtml) miscompiled by reload

2005-04-12 Thread pinskia at gcc dot gnu dot org
--- 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|