X-Git-Url: https://perl5.git.perl.org/perl5.git/blobdiff_plain/666ea1927d03c6430ac8120e8603244e721e20a0..8b87be80c126a204fd0c99804a6d4a096ae3aa6d:/cflags.SH diff --git a/cflags.SH b/cflags.SH index cc369d9..f57cf2f 100755 --- a/cflags.SH +++ b/cflags.SH @@ -17,6 +17,11 @@ case "$0" in */*) cd `expr X$0 : 'X\(.*\)/'` ;; esac +if test -f config_h.SH -a ! -f config.h; then + . ./config_h.SH + CONFIG_H=already-done +fi + warn='' # Add -Wall for the core modules iff gcc and not already -Wall @@ -30,76 +35,169 @@ Intel*) ;; # The Intel C++ plays gcc on TV but is not really it. ;; esac -# The gcc -ansi -pedantic require their own dance, too. -case "$gccversion" in -'') ;; -Intel*) ;; # Is that you, Intel C++? -*) case "$gccansipedantic" in - define) - case "$gccversion" in - [12]*) ;; # gcc versions 1 (gasp!) and 2 are not good for this. - *) case "$osname" in - # Add -ansi -pedantic only for known platforms. - aix|dec_osf|freebsd|hpux|irix|linux) - ansipedantic="-ansi -pedantic" ;; - solaris) -# Can't add -ansi for Solaris. -# Off_t/off_t is a struct in Solaris with largefiles, and with -ansi -# that struct cannot be compared with a flat integer, such as a STRLEN. -# The -ansi will also cause a lot of noise in Solaris because of: -# /usr/include/sys/resource.h:148: warning: `struct rlimit64' declared inside parameter list - ansipedantic="-pedantic" ;; - esac - for i in $ansipedantic - do - case "$ccflags" in - *$i*) ;; - *) warn="$warn $i" ;; - esac - done - case "$warn$ccflags" in - *-pedantic*) warn="$warn -DPERL_GCC_PEDANTIC" ;; - esac - ;; - esac - ;; - esac - ;; -esac +# Create a test source file for testing what options can be fed to +# gcc in this system; include a selection of most common and commonly +# hairy include files. + +cat >_cflags.c <<__EOT__ +#include "EXTERN.h" +#include "perl.h" +/* The stdio.h, errno.h, and setjmp.h should be there in any ANSI C89. */ +#include +#include +#include +/* Just in case the inclusion of perl.h did not + * pull in enough system headers, let's try again. */ +#ifdef I_STDLIB +#include +#endif +#ifdef I_STDDEF +#include +#endif +#ifdef I_STDARG +#include +#endif +#ifdef I_LIMITS +#include +#endif +#ifdef I_DIRENT +#include +#endif +#ifdef I_UNISTD +#include +#endif +#ifdef I_SYS_TYPES +#include +#endif +#ifdef I_SYS_PARAM +#include +#endif +#ifdef I_SYS_RESOURCE +#include +#endif +#ifdef I_SYS_SELECT +#include +#endif +#if defined(HAS_SOCKET) && !defined(VMS) && !defined(WIN32) /* See perl.h. */ +#include +#endif +#ifdef I_SYS_STAT +#include +#endif +#ifdef I_SYS_TIME +#include +#endif +#ifdef I_SYS_TIMES +#include +#endif +#ifdef I_SYS_WAIT +#include +#endif +/* The gcc -ansi can cause a lot of noise in Solaris because of: + /usr/include/sys/resource.h:148: warning: 'struct rlimit64' declared inside parameter list + */ +int main(int argc, char *argv[]) { + +/* Add here test code found to be problematic in some gcc platform. */ + +/* Off_t/off_t is a struct in Solaris with largefiles, and with gcc -ansi + * that struct cannot be compared in some gcc releases with a flat + * integer, such as a STRLEN. */ + + IV iv; + Off_t t0a = 2; + STRLEN t0b = 3; + int t0c = t0a == t0b; + +/* In FreeBSD 6.2 (and probably other releases too), with -Duse64bitint, + perl will use atoll(3). However, that declaration is hidden in + if we force the compiler to use -std=c89 mode. +*/ + iv = Atol("42"); + + return (!t0c && (iv == 42)) ? 0 : -1; /* Try to avoid 'unused' warnings. */ +} +__EOT__ + +stdflags='' + +# Further gcc warning options. Build up a list of options that work. +# Note that some problems may only show up with combinations of options, +# e.g. a warning might show up only with -Wall -ansi, not with either +# one individually. +# TODO: Ponder whether to migrate this back to Configure so hints files can +# tweak it. Also, be paranoid about whether results we've deduced in Configure +# (especially about things like long long, which are not in C89) will still be +# valid if we now add flags like -std=c89. -# Further gcc warning options. case "$gccversion" in '') ;; -[12]*) ;; -Intel*) ;; # Haven't we been through this already? -*) for opt in '' extra declaration-after-statement endif-labels +[12]*) ;; # gcc versions 1 (gasp!) and 2 are not good for this. +Intel*) ;; # # Is that you, Intel C++? +*) for opt in -ansi -std=c89 -W -Wextra -Wdeclaration-after-statement \ + -Wendif-labels -Wc++-compat -Wwrite-strings do case " $ccflags " in - *"-W$opt "*) ;; - *) case "`echo >_cflags.c | $cc -W$opt -c _cflags.c -o _cflags.o 2>&1`" in + *" $opt "*) ;; # Skip if already there. + *) rm -f _cflags$_exe + case "`$cc $cflags $warn $stdflags $opt _cflags.c -o _cflags$_exe 2>&1`" in *"unrecognized"*) ;; + *"implicit declaration"*) ;; # Was something useful hidden? *"Invalid"*) ;; - *) warn="$warn -W$opt" ;; + *"is valid for C"*) ;; + *) if test -x _cflags$_exe + then + case "$opt" in + -std*) stdflags="$stdflags $opt" ;; + *) warn="$warn $opt" ;; + esac + fi + ;; esac ;; esac - rm -f _cflags.c cflags.o done ;; esac +rm -f _cflags.c _cflags$_exe -# If we have g++, we cannot have the -Wdeclaration-after-statement. -# Some g++s accept it but then whine about it with every file. -case "$cc" in -*g++*) warn="`echo $warn|sed 's/-Wdeclaration-after-statement/ /'`" ;; +case "$gccversion" in +'') ;; +*) + if [ "$gccansipedantic" = "" ]; then + # If we have -Duse64bitint (or equivalent) in effect and the quadtype + # has become 'long long', gcc -pedantic becomes unbearable (moreso + # when combined with -Wall) because long long and LL and %lld|%Ld + # become warn-worthy. So let's drop the -pedantic in that case. + case "$quadtype:$sPRId64" in + "long long"*|*lld*|*Ld*) + ccflags="`echo $ccflags|sed 's/-pedantic/ /'`" + warn="`echo $warn|sed 's/-pedantic/ /'`" + ;; + esac + # Similarly, since 'long long' isn't part of C89, FreeBSD 6.2 headers + # don't declare atoll() under -std=c89, but we need it. In general, + # insisting on -std=c89 is inconsistent with insisting on using + # 'long long'. So drop -std=c89 and -ansi as well if we're using + # 'long long' as our main integral type. + case "$ivtype" in + "long long") + ccflags=`echo $ccflags|sed -e 's/-pedantic/ /' -e 's/-std=c89/ /' -e 's/-ansi/ /'` + warn=`echo $warn|sed -e 's/-pedantic/ /' -e 's/-ansi/ /'` + stdflags=`echo $stdflags|sed -e 's/-std=c89/ /'` + ;; + esac + fi + # Using certain features (like the gcc statement expressions) + # requires knowing whether -pedantic has been specified. + case "$warn$ccflags" in + *-pedantic*) warn="$warn -DPERL_GCC_PEDANTIC" ;; + esac + ;; esac -# stdflags currently unused. -stdflags='' - -extra='' - # Code to set any extra flags here. +extra='' echo "Extracting cflags (with variable substitutions)" : This section of the file will have variable substitutions done on it. @@ -116,6 +214,8 @@ warn="$warn" stdflags="$stdflags" # Extra extra. extra="$extra" +# what do executables look like? +_exe="$_exe" !GROK!THIS! @@ -220,17 +320,42 @@ for file do usersub) ;; utf8) ;; util) ;; - xsutils) ;; *) ;; esac -case "$cc" in -*g++*) ccflags="`echo $ccflags|sed 's/-Wdeclaration-after-statement/ /'`" ;; -esac + case "$cc" in + *g++*) + # Extra paranoia in case people have bad canned ccflags: + # bad in the sense that the flags are accepted by g++, + # but then whined about. + for f in -Wdeclaration-after-statement -std=c89 + do + ccflags=`echo $ccflags|sed 's/$f/ /'` + done + ;; + esac + cppflags=`echo $cppflags|sed 's/-Wdeclaration-after-statement/ /'` + + case "$cc" in + *g++*) + # Without -Wno-unused-variable g++ 4.x compiles are rather unwatchable + # because of all the warnings about Perl___notused, and g++ doesn't do + # __attribute__((unused)) (and even if at some stage it may, people do + # have older gcc installations), and ((void)x) isn't enough to silence + # the noises about XS functions not using their cv parameter, so we need + # the -Wno-unused-parameter too. + # Yes, we lose some valid warnings, but hopefully other compilers + # (like gcc) will still pick up those warnings. + for o in -Wno-unused-variable -Wno-unused-parameter + do + case "$warn" in + *$o*) ;; + *) warn="$warn $o" ;; + esac + done + ;; + esac -if test -f .patch; then - ccflags="-DPERL_PATCHNUM=`cat .patch` $ccflags" -fi : Can we perhaps use $ansi2knr here echo "$cc -c -DPERL_CORE $ccflags $stdflags $optimize $warn $extra" @@ -238,6 +363,7 @@ fi . $TOP/config.sh + # end per file behaviour done !NO!SUBS! chmod 755 cflags