X-Git-Url: https://perl5.git.perl.org/perl5.git/blobdiff_plain/c94db568e030a227855e08f584b1bff860be0cf5..fae2e96056d70d1922a44b0166c5c36a43da092d:/cflags.SH diff --git a/cflags.SH b/cflags.SH index 8f9c249..abf721f 100755 --- a/cflags.SH +++ b/cflags.SH @@ -1,3 +1,28 @@ +#!/bin/sh + +# Generate the cflags script, which is used to determine what cflags +# to pass to the compiler for compiling the core perl. +# +# This does NOT affect the XS compilation (ext, dist, cpan) +# since that uses %Config values directly. +# +# For example, since -Wall adds -Wunused-*, a bare -Wall (without +# amending that with -Wno-unused-..., or with the PERL_UNUSED_...) +# would be too much for XS code because there are too many generated +# but often unused things. +# +# We create a temporary test C program and repeatedly compile it with +# various candidate flags, and from the compiler output, determine what +# flags are supported. +# +# From this we initialise the following variables in the cflags script: +# +# $myccflags (possibly edited version of $Config{ccflags}) +# $warn +# $stdflags +# $extra +# $_exe + case $PERL_CONFIG_SH in '') if test -f config.sh; then TOP=.; @@ -11,13 +36,13 @@ case $PERL_CONFIG_SH in . $TOP/config.sh ;; esac -: This forces SH files to create target in same directory as SH file. -: This is so that make depend always knows where to find SH derivatives. +# This forces SH files to create target in same directory as SH file. +# This is so that make depend always knows where to find SH derivatives. case "$0" in */*) cd `expr X$0 : 'X\(.*\)/'` ;; esac -if test -f config_h.SH && ! test -f config.h; then +if test -f config_h.SH -a ! -f config.h; then . ./config_h.SH CONFIG_H=already-done fi @@ -66,31 +91,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: @@ -104,28 +129,50 @@ 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 -Wc++-compat +# XXX if -pedantic (or -pedantic-errors!) is ever added, +# the -Werror=declaration-after-statement can be removed +# since -std=c89 -pedantic implies the -Werror=... +*) for opt in -ansi -std=c89 -W -Wextra \ + -Werror=declaration-after-statement \ + -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 -DPERL_NO_INLINE_FUNCTIONS $ccflags $warn $stdflags $opt _cflags.c -o _cflags$_exe 2>&1`" in *"unrecognized"*) ;; + *"unknown"*) ;; + *"implicit declaration"*) ;; # Was something useful hidden? *"Invalid"*) ;; *"is valid for C"*) ;; *) if test -x _cflags$_exe @@ -154,41 +201,96 @@ case "$gccversion" in # become warn-worthy. So let's drop the -pedantic in that case. case "$quadtype:$sPRId64" in "long long"*|*lld*|*Ld*) + echo "cflags.SH: Removing -pedantic and warn because of quadtype='long long'." 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") + echo "cflags.SH: Removing -pedantic, -std=c89, and -ansi because of ivtype='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" ;; + *-pedantic*) + echo "cflags.SH: Adding -DPERL_PEDANTIC because of -pedantic." + warn="$warn -DPERL_GCC_PEDANTIC" + ;; esac ;; 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. + # + # -Werror=d-a-s option is valid for g++, by definition, + # but we remove it just for cleanliness and shorter command lines. + for f in -Wdeclaration-after-statement \ + -Werror=declaration-after-statement \ + -std=c89 + do + case "$ccflags" in + *"$f"*) + echo "cflags.SH: Removing $ccflags because of g++." + ccflags=`echo $ccflags|sed 's/$f/ /'` ;; + esac + done + ;; +esac + +for f in -Wdeclaration-after-statement -Werror=declaration-after-statement +do + case "$cppflags" in + *"$f"*) + echo "cflags.SH: Removing $f from cppflags." + cppflags=`echo $cppflags|sed 's/$f/ /'` ;; + esac +done + # 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. -: Move anything that needs config subs from !NO!SUBS! section to !GROK!THIS!. -: Protect any dollar signs and backticks that you do not want interpreted -: by putting a backslash in front. You may delete these comments. +# This section of the file will have variable substitutions done on it. +# Move anything that needs config subs from !NO!SUBS! section to !GROK!THIS!. +# Protect any dollar signs and backticks that you do not want interpreted +# by putting a backslash in front. You may delete these comments. rm -f cflags $spitshell >cflags <>cflags <<'!NO!SUBS!' case $PERL_CONFIG_SH in '') @@ -201,11 +303,17 @@ case $PERL_CONFIG_SH in echo "Can't find config.sh."; exit 1 fi . $TOP/config.sh + ccflags="$myccflags" # Restore possible edits by cflags.SH. ;; esac -: syntax: cflags [optimize=XXX] [file[.suffix]] -: displays the compiler command line for file +# syntax: cflags [optimize=XXX] [file[.suffix]] ... +# displays the proposed compiler command line for each 'file' +# +# with no file, dispalys it for all *.c files. +# The optimise=XXX arg (if present) is evalled, setting the default +# value of the $optimise variable, which is output on the command line +# (but which may be overridden for specific files below) case "X$1" in Xoptimize=*|X"optimize=*") @@ -214,11 +322,6 @@ Xoptimize=*|X"optimize=*") ;; esac -also=': ' -case $# in -1) also='echo 1>&2 " CCCMD = "' -esac - case $# in 0) set *.c; echo "The current C flags are:" ;; esac @@ -232,109 +335,39 @@ for file do *) echo $n " $file.c $c" ;; esac - : allow variables like toke_cflags to be evaluated + # allow variables like toke_cflags to be evaluated if echo $file | grep -v / >/dev/null then eval 'eval ${'"${file}_cflags"'-""}' fi - : or customize here + # 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++*) - # 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 -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" ;; + # Customization examples follow. + # + # The examples are intentionally unreachable as the '*)' case above 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. + + 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 - done - ;; -esac - -if test -f .patch; then - ccflags="-DPERL_PATCHNUM=`cat .patch` $ccflags" -fi - : Can we perhaps use $ansi2knr here + # Can we perhaps use $ansi2knr here echo "$cc -c -DPERL_CORE $ccflags $stdflags $optimize $warn $extra" - eval "$also "'"$cc -DPERL_CORE -c $ccflags $stdflags $optimize $warn $extra"' . $TOP/config.sh + # end per file behaviour done !NO!SUBS! chmod 755 cflags