path: root/config/gnu-flags

#                            -*- shell-script -*-
#
# Copyright by The HDF Group.
# Copyright by the Board of Trustees of the University of Illinois.
# All rights reserved.
#
# This file is part of HDF5.  The full HDF5 copyright notice, including
# terms governing use, modification, and redistribution, is contained in
# the COPYING file, which can be found at the root of the source code
# distribution tree, or in https://support.hdfgroup.org/ftp/HDF5/releases.
# If you do not have access to either file, you may request a copy from
# help@hdfgroup.org.


# This file should be sourced into configure if the compiler is the
# GNU gcc compiler or a derivative.  It is careful not to do anything
# if the compiler is not GNU; otherwise 'cc_flags_set' is set to 'yes'
#

# Get the compiler version in a way that works for gcc
# unless a compiler version is already known
#
#   cc_vendor:    The compiler name: gcc
#   cc_version:   Version number: 2.91.60, 2.7.2.1
#
if test "X-" = "X-$cc_flags_set"; then
    # PathScale compiler spits out gcc version string too. Need to
    # filter it out.
    # icc beginning with version 12 includes a "gcc version compatiblilty"
    # string, causing the gcc H5_CFLAGS to be erroneously added.  The line
    # "grep -v 'icc version'" causes the discarding of any output
    # containing 'icc version'.  The cc_version for icc is correctly determined
    # and flags added in the intel-flags script.
    cc_version="`$CC $CFLAGS $H5_CFLAGS -v 2>&1 | grep -v 'PathScale' |\
        grep -v 'icc version' |\
        grep 'gcc version' | sed 's/.*gcc version \([-a-z0-9\.]*\).*/\1/'`"
    cc_vendor=`echo $cc_version |sed 's/\([a-z]*\).*/\1/'`
    cc_version=`echo $cc_version |sed 's/[-a-z]//g'`
    if test X = "X$cc_vendor" -a X != "X$cc_version"; then
        cc_vendor=gcc
    fi
    if test "-" != "$cc_vendor-$cc_version"; then
        echo "compiler '$CC' is GNU $cc_vendor-$cc_version"
    fi

    # Get the compiler version numbers
    cc_vers_major=`echo $cc_version | cut -f1 -d.`
    cc_vers_minor=`echo $cc_version | cut -f2 -d.`
    cc_vers_patch=`echo $cc_version | cut -f3 -d.`
    test -n "$cc_vers_major" || cc_vers_major=0
    test -n "$cc_vers_minor" || cc_vers_minor=0
    test -n "$cc_vers_patch" || cc_vers_patch=0
fi

if test "X-gcc" = "X-$cc_vendor"; then

    ###############################
    # Architecture-specific flags #
    ###############################

    arch=
    case "$host_os-$host_cpu" in
        # FreeBSD sets the information from "uname -m" to the general machine
        # architecture, not the specific CPU for the machine, so even our
        # Pentium II Xeon server is set to "i386".  Once we know we are on a FreeBSD
        # machine, use the "sysctl" command to get the CPU hardware model.
        freebsd*-i386)
            host_cpu_model=`sysctl -n hw.model`
            case "$host_cpu_model" in
                # Hmm.. this might not catch Celerons, but it won't hurt them either...
                *Pro*|*II*|*III*|*IV*|*Athlon*)
                    # architecture-specific optimizations cause problems
                    # for some users who build binaries to be used on
                    # multiple architectures.
                    # arch="-march=i686"
                ;;
            esac
            ;;

        *-i686)
                    # architecture-specific optimizations cause problems
                    # for some users who build binaries to be used on
                    # multiple architectures.
                    # arch="-march=i686"
            ;;
    esac

    case "$host_os-$host_cpu" in
        # cygwin needs the "-std=c99" flag removed, so make
        # a specific case for Cygwin without the flag and a default
        # case to add the flag everywhere else
        cygwin-*)
            ;;

        *)
            H5_CFLAGS="$H5_CFLAGS -std=c99"
            ;;
    esac

    H5_CFLAGS="$H5_CFLAGS $arch"

    ##############
    # Production #
    ##############

    # NDEBUG is handled explicitly by the configure script
    if test $cc_vers_major -le 4; then
        PROD_CFLAGS=
    else
        PROD_CFLAGS="-fstdarg-opt"
    fi

    #########
    # Debug #
    #########

    # NDEBUG is handled explicitly by the configure script
    # -g is handled by the symbols flags
    if test $cc_vers_major -le 4; then
        DEBUG_CFLAGS=
    else
        DEBUG_CFLAGS="-ftrapv -fno-common"
    fi

    ###########
    # Symbols #
    ###########

    NO_SYMBOLS_CFLAGS="-s"
    SYMBOLS_CFLAGS="-g -fno-omit-frame-pointer"

    #############
    # Profiling #
    #############

    PROFILE_CFLAGS="-pg"

    ################
    # Optimization #
    ################
    
    if test $cc_vers_major -le 4; then
        HIGH_OPT_CFLAGS="-O3"
        DEBUG_OPT_CFLAGS=
    else
        HIGH_OPT_CFLAGS="-O3"
        DEBUG_OPT_CFLAGS="-Og"
    fi
    NO_OPT_CFLAGS="-O0"

    ############
    # Warnings #
    ############

    ###########
    # General #
    ###########

    # Note that some of the flags listed here really should be developer
    # flags (listed in a separate variable, below) but we put them here
    # because they are not raised by the current code and we'd like to
    # know if they do start showing up.
    #
    # NOTE: Don't add -Wpadded here since we can't/won't fix the (many)
    # warnings that are emitted. If you need it, add it from the
    # environment variable at configure time.
    #
    # NOTE: Disable the -Wformat-nonliteral from -Wformat=2 here and re-add
    # it to the developer flags.
    #
    H5_CFLAGS="$H5_CFLAGS -pedantic -Wall -Wextra"
    H5_CFLAGS="$H5_CFLAGS -Wcast-qual -Wconversion -Wfloat-equal"
    H5_CFLAGS="$H5_CFLAGS -Wformat=2 -Wno-format-nonliteral -Winit-self -Winvalid-pch -Wmissing-include-dirs"
    H5_CFLAGS="$H5_CFLAGS -Wshadow"
    H5_CFLAGS="$H5_CFLAGS -Wundef -Wwrite-strings"

    #
    # HDF5 code should not trigger the following warnings under any
    # circumstances, so ask the compiler to treat them as errors:
    #
    H5_ECFLAGS="$H5_ECFLAGS -Werror=bad-function-cast"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=cast-align"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=cast-function-type"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=declaration-after-statement"
#   H5_ECFLAGS="$H5_ECFLAGS -Werror=discarded-qualifiers"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=implicit-function-declaration"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=incompatible-pointer-types"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=maybe-uninitialized"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=missing-declarations"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=missing-prototypes"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=nested-externs"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=old-style-definition"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=packed"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=pointer-sign"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=pointer-to-int-cast"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=redundant-decls"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=shadow"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=strict-prototypes"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=switch"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=unused-but-set-variable"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=unused-function"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=unused-parameter"
    H5_ECFLAGS="$H5_ECFLAGS -Werror=unused-variable"

    ######################
    # Developer warnings #
    ######################

    # (suggestions from gcc, not code problems)
    # NOTE: -Wformat-nonliteral added back in here (from being disabled in H5_CFLAGS)
    DEVELOPER_WARNING_CFLAGS="-Winline -Waggregate-return -Wmissing-format-attribute -Wmissing-noreturn -Wformat-nonliteral"
    NO_DEVELOPER_WARNING_CFLAGS="-Wno-inline -Wno-aggregate-return -Wno-missing-format-attribute -Wno-missing-noreturn"

    #
    # Let's discuss:
    #
    DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wdisabled-optimization"
    DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wunused-macros"
    DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wunsafe-loop-optimizations"

    #
    # -Wswitch is helpful, but these seem a step too far---let's discuss:
    #
    DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wswitch-default"
    DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wswitch-enum"

    #######################
    # gcc 4 special cases #
    #######################

    # Disable warnings about using the 'long long' type w/ gcc 4.6 and earlier
    if test $cc_vers_major -eq 4 -a $cc_vers_minor -ge 2 -o $cc_vers_major -eq 4 -a $cc_vers_minor -le 6; then
        H5_CFLAGS="$H5_CFLAGS -Wno-long-long"
    fi

    # -Wvolatile-register-var was later incorporated into -Wall and              
    # only needs to be specified explicitly for gcc 4.2-4.3                      
    if test $cc_vers_major -eq 4 -a $cc_vers_minor -ge 2 -o $cc_vers_major -eq 4 -a $cc_vers_minor -le 3; then
        H5_CFLAGS="$H5_CFLAGS -Wvolatile-register-var"                           
    fi 

    # -Wstrict-aliasing was later incorporated into -Wall and
    # only needs to be specified explicitly for gcc 4.5-4.6
    if test $cc_vers_major -eq 4 -a $cc_vers_minor -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -le 6; then
        H5_CFLAGS="$H5_CFLAGS -Wstrict-aliasing"
    fi

    # The non-valued form of -Wstrict-overflow is used in gcc 4.2-4.4
    if test $cc_vers_major -eq 4 -a $cc_vers_minor -ge 2 -a $cc_vers_major -eq 4 -a $cc_vers_minor -le 4; then
        H5_CFLAGS="$H5_CFLAGS -Wstrict-overflow"
    fi

    # -Wvla was later incorporated into -Wpedantic and
    # only needs to be specified explicitly for gcc 4
    if test $cc_vers_major -eq 4 -a $cc_vers_minor -ge 2; then
        H5_CFLAGS="$H5_CFLAGS -Wvla"
    fi

    #############################
    # Version-specific warnings #
    #############################

    # gcc 4.3                                                                    
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 3; then
        H5_CFLAGS="$H5_CFLAGS -Wlogical-op"
        #
        # Lots of noise, questionable benefit:
        #
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wlarger-than=2560"
        #
    fi 

    # gcc 4.4
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 4; then
        H5_CFLAGS="$H5_CFLAGS -Wsync-nand -Wpacked-bitfield-compat"
        #
        # Lots of noise, questionable benefit:
        #
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wframe-larger-than=16384"
        #
    fi

    # gcc 4.5
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 5; then
        H5_CFLAGS="$H5_CFLAGS -Wstrict-overflow=5 -Wunsuffixed-float-constants"
        #
        # -Wjump-misses-init makes lots of noise for a questionable benefit.
        # Can jumping over an initialization in C cause any harm, if
        # the variable is never *used* before it has been initialized?
        #
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wjump-misses-init"
        #
    fi

    # gcc 4.6
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 6; then
        H5_CFLAGS="$H5_CFLAGS -Wdouble-promotion -Wtrampolines"
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wsuggest-attribute=const"
        NO_DEVELOPER_WARNING_CFLAGS="$NO_DEVELOPER_WARNING_CFLAGS -Wno-suggest-attribute=const"
    fi

    # gcc 4.7
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 7; then
        #
        # It's not clear that -Wvector-operation-performance warnings are
        # actionable.
        #
        # -Wstack-usage=8192 warnings need better justification;
        # if justifiable, should be enabled on a branch and swept up there
        # before burdening the whole development team.
        #
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wstack-usage=8192"
	DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wvector-operation-performance"

        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wsuggest-attribute=pure -Wsuggest-attribute=noreturn"
        NO_DEVELOPER_WARNING_CFLAGS="$NO_DEVELOPER_WARNING_CFLAGS -Wno-suggest-attribute=pure -Wno-suggest-attribute=noreturn"
    fi

    # gcc 4.8
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 8; then
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wsuggest-attribute=format"
        NO_DEVELOPER_WARNING_CFLAGS="$NO_DEVELOPER_WARNING_CFLAGS -Wno-suggest-attribute=format"
    fi

    # gcc 4.9
    if test $cc_vers_major -ge 5 -o $cc_vers_major -eq 4 -a $cc_vers_minor -ge 9; then
        H5_CFLAGS="$H5_CFLAGS -Wdate-time"
    fi

    # gcc 5
    if test $cc_vers_major -ge 5; then
        H5_CFLAGS="$H5_CFLAGS -Warray-bounds=2 -Wc99-c11-compat"
    fi

    # gcc 6
    if test $cc_vers_major -ge 6; then
        H5_CFLAGS="$H5_CFLAGS -Wunused-const-variable -Whsa -Wnormalized"
        #
        # Unacceptably noisy on HDF5 right now.
        #
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wnull-dereference"
        #
        # Careful!  -Wduplicated-cond, combined with HDF5's heavy use of
        # macros, can make a lot of noise.
        #
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wduplicated-cond"
    fi

    # gcc 7
    if test $cc_vers_major -ge 7; then
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wstringop-overflow=2"
        H5_CFLAGS="$H5_CFLAGS -Walloc-zero -Walloca -Wduplicated-branches -Wformat-overflow=2 -Wformat-truncation=2 -Wimplicit-fallthrough=5 -Wrestrict"
    fi

    # gcc 8
    if test $cc_vers_major -ge 8; then
        DEVELOPER_WARNING_CFLAGS="$DEVELOPER_WARNING_CFLAGS -Wstringop-overflow=4 -Wsuggest-attribute=cold -Wsuggest-attribute=malloc"
        NO_DEVELOPER_WARNING_CFLAGS="$NO_DEVELOPER_WARNING_CFLAGS -Wno-suggest-attribute=cold -Wno-suggest-attribute=malloc"
        H5_CFLAGS="$H5_CFLAGS -Wattribute-alias -Wcast-align=strict -Wshift-overflow=2"
    fi

    # gcc 9
    if test $cc_vers_major -ge 9; then
        H5_CFLAGS="$H5_CFLAGS -Wattribute-alias=2 -Wmissing-profile"
    fi

    #################
    # Flags are set #
    #################
    cc_flags_set=yes
fi

# Clear cc info if no flags set
if test "X$cc_flags_set" = "X"; then
    cc_vendor=
    cc_version=
fi