Re: Re: [AD] Remove -pipe

[allegrovp@xxxxxxxxxx]

Well, I did some timings (make clean, then time make):

with -pipe:

real    13m34.912s
user    12m16.160s
sys     1m9.550s

without -pipe:

real    13m39.584s
user    12m14.490s
sys     1m13.590s


So -pipe is not having much effect at all. I recall it having more effect, but I had a memory shortage at the time, and compiles temps were hitting the disk.

Anyway, if such patches get reverted, the following should be added, as it allows one to customize CFLAGS and LDFLAGS without adding complexity to the configure machinery.
Can't attach a patch from this account, so it goes in the body. It's tiny anyway.


--- makefile.in.override        Sun May 16 21:15:25 2004
+++ makefile.in Sun May 16 21:16:25 2004
@@ -61,12 +61,12 @@
 INCLUDES = -I. -Iinclude -Iinclude/allegro -I$(srcdir)/include -I$(srcdir)/include/allegro
 DEFS = @DEFS@
 CPPFLAGS = @CPPFLAGS@
-LDFLAGS = @LDFLAGS@
+override LDFLAGS += @LDFLAGS@
 PROG_LDFLAGS = @PROG_LDFLAGS@
 COMPILE = $(CC) $(DEFS) $(INCLUDES) $(CPPFLAGS) $(CFLAGS)
 LINK = $(CC) $(PROG_LDFLAGS)

-CFLAGS = @CFLAGS@
+override CFLAGS += @CFLAGS@
 ALLEGRO_DEBUG_CFLAGS = @ALLEGRO_DEBUG_CFLAGS@
 ALLEGRO_PROFILE_CFLAGS = @ALLEGRO_PROFILE_CFLAGS@
 ALLEGRO_SHAREDLIB_CFLAGS = @ALLEGRO_SHAREDLIB_CFLAGS@



Vincent