Hi,

> -PASS: abi/header_cxxabi.c (test for excess errors)
> +FAIL: abi/header_cxxabi.c (test for excess errors)
> 
> FAIL: abi/header_cxxabi.c (test for excess errors)
> Excess errors:
> /var/gcc/regression/trunk/11-gcc/build/i386-pc-solaris2.11/libstdc++-v3/include/i386-pc-solaris2.11/bits/c++config.h:167:1:
>  error: unknown type name 'namespace'
> /var/gcc/regression/trunk/11-gcc/build/i386-pc-solaris2.11/libstdc++-v3/include/i386-pc-solaris2.11/bits/c++config.h:168:1:
>  error: expected '=', ',', ';', 'asm' or '__attribute__' before '{' token
> 
> which is pretty obvious given that this test is supposed to be compiled
> as C :-)

To be honest, I'm not at all sure to understand what's going on here, maybe we 
should return to the fail later.

> I guess the patch is ok now?

Yes. Nice that Jon replied in the meanwhile and clarified the undefined 
behavior issue: for the time being I think we can keep the __cplusplus checks, 
should also help during this testing period. We can certainly clean up the 
thing later. Maybe you could add a comment somewhere summarizing what Jon wrote.

By the way I noticed only today (sorry, I'm using a phone ;) that all the new 
configury work impacts only Solaris configuration, thus in general we are on 
pretty safe ground

Paolo

Reply via email to