X-Git-Url: https://perl5.git.perl.org/perl5.git/blobdiff_plain/a07cd53d52eddd3373b7f4babb55144d44a1613d..fb73dc2fcc96a4a5edf1daa66f0114b4b2b6a196:/cflags.SH diff --git a/cflags.SH b/cflags.SH index 1c542a0..899c465 100755 --- a/cflags.SH +++ b/cflags.SH @@ -1,3 +1,5 @@ +#!/bin/sh + case $PERL_CONFIG_SH in '') if test -f config.sh; then TOP=.; @@ -17,6 +19,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 @@ -61,31 +68,31 @@ cat >_cflags.c <<__EOT__ #ifdef I_UNISTD #include #endif -#ifdef I_SYSTYPES +#ifdef I_SYS_TYPES #include #endif -#ifdef I_SYSPARAM +#ifdef I_SYS_PARAM #include #endif -#ifdef I_SYSRESOURCE +#ifdef I_SYS_RESOURCE #include #endif -#ifdef I_SYSSELECT +#ifdef I_SYS_SELECT #include #endif #if defined(HAS_SOCKET) && !defined(VMS) && !defined(WIN32) /* See perl.h. */ #include #endif -#ifdef I_SYSSTAT +#ifdef I_SYS_STAT #include #endif -#ifdef I_SYSTIME +#ifdef I_SYS_TIME #include #endif -#ifdef I_SYSTIMES +#ifdef I_SYS_TIMES #include #endif -#ifdef I_SYSWAIT +#ifdef I_SYS_WAIT #include #endif /* The gcc -ansi can cause a lot of noise in Solaris because of: @@ -99,29 +106,47 @@ int main(int argc, char *argv[]) { * 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; - return 0; +/* 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. +# 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. + case "$gccversion" in '') ;; [12]*) ;; # gcc versions 1 (gasp!) and 2 are not good for this. Intel*) ;; # # Is that you, Intel C++? -*) for opt in -ansi -pedantic -std=c89 -W -Wextra -Wdeclaration-after-statement -Wendif-labels +*) for opt in -ansi -std=c89 -W -Wextra -Wdeclaration-after-statement \ + -Wendif-labels -Wc++-compat -Wwrite-strings do case " $ccflags " in *" $opt "*) ;; # Skip if already there. *) rm -f _cflags$_exe - case "`$cc $cflags $opt _cflags.c -o _cflags$_exe 2>&1`" in + case "`$cc $cflags $warn $stdflags $opt _cflags.c -o _cflags$_exe 2>&1`" in *"unrecognized"*) ;; + *"implicit declaration"*) ;; # Was something useful hidden? *"Invalid"*) ;; + *"is valid for C"*) ;; *) if test -x _cflags$_exe then case "$opt" in @@ -138,28 +163,41 @@ Intel*) ;; # # Is that you, Intel C++? 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. -# This removal is also done "later", in cflags run time. -case "$cc" in -*g++*) warn="`echo $warn|sed 's/-Wdeclaration-after-statement/ /'`" ;; -esac - -# 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/ /'`" +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 -case "$warn$ccflags" in -*-pedantic*) warn="$warn -DPERL_GCC_PEDANTIC" ;; -esac - # Code to set any extra flags here. extra='' @@ -178,6 +216,8 @@ warn="$warn" stdflags="$stdflags" # Extra extra. extra="$extra" +# what do executables look like? +_exe="$_exe" !GROK!THIS! @@ -235,87 +275,60 @@ for file do : or customize here case "$file" in - DB_File) ;; - GDBM_File) ;; - NDBM_File) ;; - ODBM_File) ;; - POSIX) ;; - SDBM_File) ;; - av) ;; - byterun) ;; - deb) ;; - dl) ;; - doio) ;; - doop) ;; - dump) ;; - globals) ;; - gv) ;; - hv) ;; - locale) ;; - madly) ;; - main) ;; - malloc) ;; - mg) ;; - miniperlmain) ;; - numeric) ;; - op) ;; - opmini) ;; - pad) ;; - perl) ;; - perlapi) ;; - perlmain) ;; - perly) ;; - pp) ;; - pp_ctl) ;; - pp_hot) ;; - pp_pack) ;; - pp_sort) ;; - pp_sys) ;; - regcomp) ;; - regexec) ;; - run) ;; - scope) ;; - sv) ;; - taint) ;; - toke) ;; - universal) ;; - usersub) ;; - utf8) ;; - util) ;; - xsutils) ;; *) ;; - esac -case "$cc" in -*g++*) - # We need to remove this also in here (removed also earlier in cflags.SH). - ccflags="`echo $ccflags|sed 's/-Wdeclaration-after-statement/ /'`" - ;; -esac + *) : Customization examples follow: ;; + av) ccflags=`echo $ccflags | sed -e s/-pipe//` ;; + deb) ccflags="$ccflags -fno-jump-tables" ;; + hv) warn=`echo $warn | sed -e s/-Wextra//` ;; + toke) optimize=-O0 ;; + esac -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" ;; + : The examples are intentionally unreachable as the '*)' case always + : matches. To use them, move before the '*)' and edit as appropriate. + : It is not a good idea to set ccflags to an absolute value here, as it + : often contains general -D defines which are needed for correct + : compilation. It is better to edit ccflags as shown, using interpolation + : to add flags, or sed to remove flags. + + + 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 + *clang) + # clang complains a lot about -Wunused-value which are not fixable + warn="$warn -Wno-unused-value" + ;; + *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 - 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" @@ -323,6 +336,7 @@ fi . $TOP/config.sh + # end per file behaviour done !NO!SUBS! chmod 755 cflags