On Friday 03 August 2007 21:35:36 chromatic wrote:
> r20454 doesn't help. Of course, it looks like that's papering over the
> fact that the noreturn attribute expands (for me on x86 Linux, that is) to:
>
> #define PARROT_DOES_NOT_RETURN /[EMAIL PROTECTED]@*/
> __attribute__noreturn__
>
> ... when it looks like it should be:
>
> #define PARROT_DOES_NOT_RETURN /[EMAIL PROTECTED]@*/
> __attribute__((noreturn))
>
> So I'm not sure what went weird.
Ahh, this patch fixes things for me. My gcc is also available as cc (and I'm
using ccache), so checking that it matches the literal string 'gcc' doesn't
work. Explicitly checking that it has a GCC version number works better.
I don't promise that this is the *correct* solution, but it's a better one.
(Thank goodness and everyone who's coded and refactored the configuration
system for the ability to run a configure step verbosely!)
-- c
=== config/auto/attributes.pm
==================================================================
--- config/auto/attributes.pm (revision 5196)
+++ config/auto/attributes.pm (local)
@@ -63,7 +63,7 @@
my $cc = $conf->option_or_data( 'cc' );
$verbose and print " cc: $cc$/";
- if ( $cc =~ /gcc/ ) {
+ if ( $conf->data->get( 'gccversion' ) ) {
cc_gen('config/auto/gcc/test_c.in');
}
elsif ( $cc eq 'cl' ) {
