Re: [AD] msvc7 compiler options

[aj <aj@xxxxxxxxxx>]

At 01:17 AM 4/02/2005, you wrote:
> aj wrote:
> > can you provide me with a URL as my MSDN research has not found this.
> > in the VS IDE, they are on the same menu option, so its either on of the 
> > other.. and the way it looks is that -Ox is "Full Optimization" which 
> > implies fast.
>
> "In general, /O2 should be preferred over /Ox" is here:
> http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vccore/html/_core_.2f.Ox.asp
>
> "O2 is /Og /Oi /Ot /Oy /Ob2 /Gs /GF /Gy" is here:
> http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vccore/html/_core_.2f.Ox.asp
>
> "Ox is /Og /Oi /Ot /Oy /Ob2 /Gs" is visible after running "cl /?" on the 
> command line


OK,  im convinced..  /O2 looks better.
lets change it to that.

attached is my amended patch.
diff is against the original 4.1.18 (not the current CVS).

only thing changed since my last patch  of the same name is the change from 
-Ox to -O2


aj.

--- ..\allegro_4_1_18__original\makefile.vc	2004-11-24 14:09:00.000000000 +1100
+++ makefile.vc	2005-02-03 03:09:12.000000000 +1100
@@ -1,4 +1,4 @@
-#
+#   4.1.18
 #  Rules for building the Allegro library with MSVC. This file is included
 #  by the primary makefile, and should not be used directly.
 #
@@ -163,7 +163,11 @@
 
 # -------- optimised build --------
 
+ifdef COMPILER_MSVC7
+CFLAGS = $(WFLAGS) -Gd -O2 -GB -GL $(RUNTIME_FLAGS)
+else
 CFLAGS = $(WFLAGS) -Gd -Ox -GB $(RUNTIME_FLAGS)
+endif
 SFLAGS = -Wall
 LFLAGS = -release