This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
cflags.SH: Whitespace only.
[perl5.git] / cflags.SH
index af1b43b..c166db3 100755 (executable)
--- a/cflags.SH
+++ b/cflags.SH
@@ -1,3 +1,17 @@
+#!/bin/sh
+
+# Generate the cflags script, which is used to determine what cflags
+# to pass to the compiler.
+# 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:
+#
+#   $warn
+#   $stdflags
+#   $extra
+#   $_exe
+
 case $PERL_CONFIG_SH in
 '')
        if test -f config.sh; then TOP=.;
@@ -11,8 +25,8 @@ 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
@@ -66,31 +80,31 @@ cat >_cflags.c <<__EOT__
 #ifdef I_UNISTD
 #include <unistd.h>
 #endif
-#ifdef I_SYSTYPES
+#ifdef I_SYS_TYPES
 #include <sys/types.h>
 #endif
-#ifdef I_SYSPARAM
+#ifdef I_SYS_PARAM
 #include <sys/param.h>
 #endif
-#ifdef I_SYSRESOURCE
+#ifdef I_SYS_RESOURCE
 #include <sys/resource.h>
 #endif
-#ifdef I_SYSSELECT
+#ifdef I_SYS_SELECT
 #include <sys/select.h>
 #endif
 #if defined(HAS_SOCKET) && !defined(VMS) && !defined(WIN32) /* See perl.h. */
 #include <sys/socket.h>
 #endif
-#ifdef I_SYSSTAT
+#ifdef I_SYS_STAT
 #include <sys/stat.h>
 #endif
-#ifdef I_SYSTIME
+#ifdef I_SYS_TIME
 #include <sys/time.h>
 #endif
-#ifdef I_SYSTIMES
+#ifdef I_SYS_TIMES
 #include <sys/times.h>
 #endif
-#ifdef I_SYSWAIT
+#ifdef I_SYS_WAIT
 #include <sys/wait.h>
 #endif
 /* The gcc -ansi can cause a lot of noise in Solaris because of:
@@ -104,28 +118,46 @@ 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 <stdlib.h>
+   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
+*)  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 -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,15 +186,110 @@ 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.
+  for f in -Wdeclaration-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
+do
+  case "$cppflags" in
+  *"$f"*)
+    echo "cflags.SH: Removing $f from cppflags."
+    cppflags=`echo $cppflags|sed 's/$f/ /'` ;;
+  esac
+done
+
+# -Wall includes -Wunused-value and -Wunused-parameter, which may be too much
+# with some compilers.
+#
+# XXX this is related to the unfortunate fact that bare -Wall (without
+# amending -Wno-unused-...) is too much for XS code because of all the
+# often generated but unused things.
+#
+case "$ccflags$warn" in
+*-Wall*)
+  is_clang=undef
+  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 f in -Wno-unused-variable -Wno-unused-parameter
+    do
+      case "$warn" in
+      *"$f"*) ;;
+      *)
+        echo "cflags.SH: Adding $f because of g++."
+        warn="$warn $f" ;;
+      esac
+    done
+    ;;
+  *clang*) is_clang=define ;;
+  *) # clang may not be called clang, it may be called cc.
+    case "`$cc -v 2>&1`" in
+    *clang*) is_clang=define ;;
+    esac ;;
+  esac
+
+  case "$is_clang" in
+  define)
+    for f in -Wno-unused-value
+    do
+      case "$warn" in
+      *"$f"*) ;;
+      *)
+        echo "cflags.SH: Adding $f because of clang."
+        warn="$warn $f" ;;
+      esac
+    done
+    ;;
   esac
   ;;
 esac
@@ -171,24 +298,32 @@ esac
 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 <<!GROK!THIS!
 $startsh
 
+# !!!!!!!   DO NOT EDIT THIS FILE   !!!!!!!
+
+# This file is generated by cflags.SH
+
+# Used to restore possible edits by cflags.SH.
+myccflags="$ccflags"
 # Extra warnings, used e.g. for gcc.
 warn="$warn"
 # Extra standardness.
 stdflags="$stdflags"
 # Extra extra.
 extra="$extra"
+# what do executables look like?
+_exe="$_exe"
 
 !GROK!THIS!
 
-: In the following dollars and backticks do not need the extra backslash.
+# In the following dollars and backticks do not need the extra backslash.
 $spitshell >>cflags <<'!NO!SUBS!'
 case $PERL_CONFIG_SH in
 '')
@@ -201,11 +336,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 +355,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 +368,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