Re: [tree-ssa] Merge results as of 2003-05-06

[Jason Merrill <jason at redhat dot com>]

On Tue, 6 May 2003 23:20:01 -0400, Diego Novillo <dnovillo@redhat.com> wrote:

> 20030506/g++.sum:
>
> FAIL: g++.dg/compat/eh/new1 x_tst.o compile
> FAIL: g++.dg/abi/cookie2.C (test for excess errors)
> FAIL: g++.dg/eh/new1.C (test for excess errors)
> FAIL: g++.dg/init/new5.C (test for excess errors)
> FAIL: g++.old-deja/g++.abi/arraynew.C (test for excess errors)
> FAIL: g++.old-deja/g++.brendan/new3.C (test for excess errors)
> FAIL: g++.old-deja/g++.jason/vecdel.C (test for excess errors)
> FAIL: g++.old-deja/g++.law/ctors15.C (test for excess errors)
> FAIL: g++.old-deja/g++.mike/eh46.C (test for excess errors)
> FAIL: g++.old-deja/g++.oliva/delete2.C (test for excess errors)
> FAIL: g++.old-deja/g++.oliva/delete3.C (test for excess errors)
> FAIL: g++.old-deja/g++.other/new3.C (test for excess errors)
> FAIL: g++.old-deja/g++.pt/crash49.C (test for excess errors)
>
>     Jason, all these fail while trying to add a temp variable in
>     gimple_add_tmp_var.  The variable we are trying to add
>     already has a TREE_CHAIN, which triggers an abort.

Fixed.

> FAIL: g++.dg/eh/cond1.C (test for excess errors)
>
>     This is the problem I mentioned in
>     http://gcc.gnu.org/ml/gcc/2003-04/msg00386.html about the
>     gimplifier trying to put the call to __cxa_throw on the RHS
>     of an assignment.  The patch in that URL fixes the test case
>     but I'm not sure that it is the correct one.

Fixed.

> FAIL: g++.dg/eh/cleanup1.C (test for excess errors)
>
>     This causes the compiler to enter an infinite loop in
>     function.c:preserve_temp_slots.  The linked list temp_slots
>     is circular.  The program compiles fine with
>     -fdisable-simple.

I don't think it's circular, I think it's just unimaginably huge.  I think
we aren't currently dealing with temp slots properly in GENERIC, so they
build up instead of being reused.  Investigating.

Jason