diff options
| author | Quincey Koziol <koziol@hdfgroup.org> | 2004-10-12 16:11:27 (GMT) |
|---|---|---|
| committer | Quincey Koziol <koziol@hdfgroup.org> | 2004-10-12 16:11:27 (GMT) |
| commit | 86d0132279cdcb7bc9b793e94509b0abd5b3147e (patch) | |
| tree | f5882a1062877c7903340d70be8d842efc770f3c /config/gnu-flags | |
| parent | 3b057e2f232d99b9322a480742a37624b54eb8aa (diff) | |
| download | hdf5-86d0132279cdcb7bc9b793e94509b0abd5b3147e.zip hdf5-86d0132279cdcb7bc9b793e94509b0abd5b3147e.tar.gz hdf5-86d0132279cdcb7bc9b793e94509b0abd5b3147e.tar.bz2 | |
[svn-r9407] Purpose:
Code cleanup/bug fix
Description:
Remove the undocumented "$NOFP" shell variable which would allow production
builds without removing the stack frame. This was cascading between C and
FORTRAN builds and causing problems.
Solution:
Hard-code removing the stack frame for now. Users would have needed to
read the configure scripts anyway, so there's not really much benefit to the
extra knob. Users who need this functionality (production builds with stack
frames) are debugging probably, so they will need to modify the script and
build their own version of the library now.
Platforms tested:
Linux 2.4 (verbena)
Too minor to require h5committest
Diffstat (limited to 'config/gnu-flags')
| -rw-r--r-- | config/gnu-flags | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/config/gnu-flags b/config/gnu-flags index 8e543db..cb2e028 100644 --- a/config/gnu-flags +++ b/config/gnu-flags @@ -125,13 +125,9 @@ case "$cc_vendor-$cc_version" in CFLAGS="$CFLAGS $arch -ansi -pedantic -Wall -W -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wcast-align -Wwrite-strings -Wconversion -Wsign-compare -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wredundant-decls -Wnested-externs -Winline" # Production - - # Omit frame pointer for optimized code? - NOFP=${NOFP:=-fomit-frame-pointer} - case "$cc_vendor-$cc_version" in gcc-2.95.[34]) - PROD_CFLAGS="-O3 $NOFP" + PROD_CFLAGS="-O3" ;; gcc-3.[0-4]*) # The optimization level is reduced for gcc 3.* due to problems @@ -140,18 +136,18 @@ case "$cc_vendor-$cc_version" in # failures for various integer types -> long long conversions in the # test/dtypes test). Perhaps later versions of gcc will fix this # bug... - QAK - 2003/10/20 - PROD_CFLAGS="-O $NOFP" + PROD_CFLAGS="-O" ;; gcc-3*) # Be optimistic about future versions of gcc.. :-) - QAK - 2003/10/20 - PROD_CFLAGS="-O3 $NOFP" + PROD_CFLAGS="-O3" ;; *) - PROD_CFLAGS="-O $NOFP" + PROD_CFLAGS="-O" ;; esac - PROD_CFLAGS="$PROD_CFLAGS -finline-functions" + PROD_CFLAGS="$PROD_CFLAGS -fomit-frame-pointer -finline-functions" PROD_CPPFLAGS= # Debug |