mirror of
https://github.com/HDFGroup/hdf5.git
synced 2024-12-15 07:40:23 +08:00
174f4275ba
* Avoid maintenance headaches: delete the `noerror-` variants of the compiler flags files, since they essentially duplicate the `error-` files modulo the replacement of `-Werror=` with `-W` and any changes in comments. (I verified the duplication with a script.) For autoconf, reinstate the use of the `demote_errors` shell function to derive the `noerror-` content from the `error-` content. `demote_errors` replaces `-Werror=` with `-W` when `WARNINGS_AS_ERRORS` is `no`. Slightly reorder `configure.ac` so that the setting of `WARNINGS_AS_ERRORS` takes effect before the `error-` files are sourced. * Take a stab at updating the CMake files to match the changes I made to the autoconf files to remove `noerror-` files. I'm not much of a CMake user so these changes are quite rough. Looks like the duplication can be reduced with the introduction of a new macro. * Delete `noerror-` files from the MANIFEST. * Reduce duplication in the CMake files: perform the HDF5_ENABLE_WARNINGS_AS_ERRORS test once in the ADD_H5_FLAGS macro. * Add a release note.
211 lines
6.3 KiB
Bash
211 lines
6.3 KiB
Bash
# -*- 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://www.hdfgroup.org/licenses.
|
|
# 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
|
|
# Clang compiler or a derivative. It is careful not to do anything
|
|
# if the compiler is not Clang; otherwise 'cc_flags_set' is set to 'yes'
|
|
#
|
|
|
|
#
|
|
# For now, do not promote any warnings to errors.
|
|
#
|
|
WARNINGS_AS_ERRORS_DFLT=no
|
|
|
|
#
|
|
# This filter rewrites -Werror= as -W, in that way demoting warnings
|
|
# promoted to errors back to warnings, if WARNINGS_AS_ERRORS is no.
|
|
#
|
|
demote_errors()
|
|
{
|
|
if [ ${WARNINGS_AS_ERRORS:-${WARNINGS_AS_ERRORS_DFLT}} = no ]; then
|
|
sed 's,-Werror=,-W,g'
|
|
else
|
|
cat
|
|
fi
|
|
}
|
|
|
|
#
|
|
# Prepend `$srcdir/config/clang-warnings/` to the filename suffix(es) given as
|
|
# subroutine argument(s), remove comments starting with # and ending
|
|
# at EOL, replace spans of whitespace (including newlines) with spaces,
|
|
# and re-emit the file(s) thus filtered on the standard output stream.
|
|
#
|
|
load_clang_arguments()
|
|
{
|
|
set -- $(for arg; do
|
|
sed 's,#.*$,,' $srcdir/config/clang-warnings/${arg} | demote_errors
|
|
done)
|
|
IFS=' ' echo "$*"
|
|
}
|
|
# Get the compiler version in a way that works for clang
|
|
# unless a compiler version is already known
|
|
#
|
|
# cc_vendor: The compiler name: clang
|
|
# cc_version: Version number: 6.0.0, 7.3.0, ... 10.0.1
|
|
#
|
|
if test "X-" = "X-$cc_flags_set"; then
|
|
# clang -v will return version number following "clang" on Linux machines,
|
|
# but on Xcode the version number will follow "Apple LLVM version"
|
|
# Note that the Xcode reported LLVM version doesn't match the canonical
|
|
# LLVM version, so you'll need to do different version checks for
|
|
# Xcode.
|
|
cc_version="`$CC $CFLAGS $H5_CFLAGS -v 2>&1 |\
|
|
grep 'clang version' | sed 's/.*clang version \([-a-z0-9\.]*\).*/\1/'`"
|
|
if test -n "$cc_version"; then
|
|
cc_vendor="clang"
|
|
else
|
|
cc_version="`$CC $CFLAGS $H5_CFLAGS -v 2>&1 |\
|
|
grep 'Apple LLVM version' | sed 's/.*Apple LLVM version \([-a-z0-9\.]*\).*/\1/'`"
|
|
if test -n "$cc_version"; then
|
|
cc_vendor="Apple LLVM"
|
|
fi
|
|
fi
|
|
if test "X-" != "X-$cc_version"; then
|
|
|
|
# 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
|
|
fi
|
|
|
|
if test "X-clang" = "X-$cc_vendor" -o "X-Apple LLVM" = "X-$cc_vendor"; then
|
|
echo "compiler '$CC' is $cc_vendor-$cc_version"
|
|
###############################
|
|
# 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
|
|
PROD_CFLAGS=
|
|
|
|
#########
|
|
# Debug #
|
|
#########
|
|
|
|
# NDEBUG is handled explicitly by the configure script
|
|
# -g is handled by the symbols flags
|
|
DEBUG_CFLAGS="-ftrapv -fno-common"
|
|
|
|
###########
|
|
# Symbols #
|
|
###########
|
|
|
|
NO_SYMBOLS_CFLAGS=
|
|
SYMBOLS_CFLAGS="-g"
|
|
|
|
#############
|
|
# Profiling #
|
|
#############
|
|
|
|
PROFILE_CFLAGS="-pg"
|
|
|
|
################
|
|
# Optimization #
|
|
################
|
|
|
|
HIGH_OPT_CFLAGS="-O3"
|
|
NO_OPT_CFLAGS="-O0"
|
|
|
|
# -Og is only understood by clang 4+ and Xcode 9+
|
|
# Otherwise use -O1 (which is what -Og usually equates to)
|
|
if test "X-clang" = "X-$cc_vendor" -a $cc_vers_major -ge 4 -o "X-Apple LLVM" = "X-$cc_vendor" -a $cc_vers_major -ge 9; then
|
|
DEBUG_OPT_CFLAGS="-Og"
|
|
else
|
|
DEBUG_OPT_CFLAGS="-O1"
|
|
fi
|
|
|
|
############
|
|
# Warnings #
|
|
############
|
|
|
|
###########
|
|
# General #
|
|
###########
|
|
|
|
H5_CFLAGS="$H5_CFLAGS $(load_clang_arguments general)"
|
|
H5_ECFLAGS="$H5_ECFLAGS $(load_clang_arguments error-general)"
|
|
|
|
######################
|
|
# Developer warnings #
|
|
######################
|
|
|
|
NO_DEVELOPER_WARNING_CFLAGS=$(load_clang_arguments no-developer-general)
|
|
DEVELOPER_WARNING_CFLAGS=$(load_clang_arguments developer-general)
|
|
|
|
#############################
|
|
# Version-specific warnings #
|
|
#############################
|
|
|
|
|
|
#################
|
|
# 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
|
|
|