Skip to content

Commit

Permalink
Don't put all of $CFLAGS into $SRC_CC_OPTS
Browse files Browse the repository at this point in the history
Instead, we just put the flags we need in there (e.g. -m64 on OS X 64).
This fixes a problem found by Simon M, where we were compiling
everything with -g, leading to a bloated RTS.
  • Loading branch information
igfoo committed Jul 16, 2009
1 parent caf2112 commit 17c5559
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -175,6 +175,9 @@ then
fi
AC_SUBST([WithGhc])

dnl ** Without optimization some INLINE trickery fails for GHCi
SRC_CC_OPTS="-O"

dnl--------------------------------------------------------------------
dnl * Choose host(/target/build) platform
dnl--------------------------------------------------------------------
Expand Down Expand Up @@ -244,6 +247,8 @@ x86_64-apple-darwin)
soext='.dylib'
CFLAGS="$CFLAGS -m64"
LDFLAGS="$LDFLAGS -m64"
SRC_CC_OPTS="$SRC_CC_OPTS -m64"
SRC_LD_OPTS="$SRC_LD_OPTS -m64"
;;
esac

Expand Down Expand Up @@ -551,12 +556,6 @@ FP_GCC_EXTRA_FLAGS
dnl ** figure out how to invoke cpp directly (gcc -E is no good)
AC_PROG_CPP

dnl ** Without optimization some INLINE trickery fails for GHCi
SRC_CC_OPTS="-O"

SRC_CC_OPTS="$SRC_CC_OPTS $CFLAGS"
SRC_LD_OPTS="$SRC_LD_OPTS $LDFLAGS"

AC_SUBST(SRC_CC_OPTS)
AC_SUBST(SRC_LD_OPTS)

Expand Down

0 comments on commit 17c5559

Please sign in to comment.