[oss-devel] [PATCH] Enable optimization in userland compile

[Yair K.]

Hi,

  The attached patch enables using the "-O" optimization level for userland 
(previously it didn't use optimizations at all). The reason is that gcc gives 
better diagnostics when it's enabled compared with -O0 default, and that it may 
unhide bugs (e.g. http://4front-tech.com/forum/viewtopic.php?p=5964#5964 was 
always visible at -O2, but otherwise bit only some configurations).

Yours,
        Yair K.

diff -r 124841211ded cmd/osstest/wavedata.c
--- a/cmd/osstest/wavedata.c	Fri Jun 13 03:49:38 2008 +0300
+++ b/cmd/osstest/wavedata.c	Fri Jun 13 12:28:24 2008 +0300
@@ -46,8 +46,8 @@
 
   int channels = 1;
   int p = 12, outp = 0;
-  int nBlockAlign;
-  int wSamplesPerBlock, wNumCoeff;
+  int nBlockAlign = 2048;
+  int wSamplesPerBlock = 2036, wNumCoeff = 7;
   int nib;
   int ppp;
 
diff -r 124841211ded lib/libOSSlib/Makefile
--- a/lib/libOSSlib/Makefile	Fri Jun 13 03:49:38 2008 +0300
+++ b/lib/libOSSlib/Makefile	Fri Jun 13 12:28:24 2008 +0300
@@ -1,7 +1,7 @@
 OBJS=guslib.o opl3lib.o libmain.o midiparser.o play_event.o
 INSTALLLIB=/lib
 LDFLAGS=-shared -fPIC
-CFLAGS=-fPIC
+CFLAGS=-O -fPIC
 #LDFLAGS=-Bdynamic
 AROPTS=rc
 AR=ar
diff -r 124841211ded lib/libsalsa/Makefile
--- a/lib/libsalsa/Makefile	Fri Jun 13 03:49:38 2008 +0300
+++ b/lib/libsalsa/Makefile	Fri Jun 13 12:28:24 2008 +0300
@@ -2,7 +2,7 @@
 
 OSSLIBDIR=/usr/lib/oss
 
-CFLAGS=-I../../include -I../libOSSlib
+CFLAGS=-O -I../../include -I../libOSSlib
 
 all: libsalsa.la
 
diff -r 124841211ded setup/srcconf.c
--- a/setup/srcconf.c	Fri Jun 13 03:49:38 2008 +0300
+++ b/setup/srcconf.c	Fri Jun 13 12:28:24 2008 +0300
@@ -725,7 +725,7 @@
 
   fprintf (f, "THISOS=%s\n", this_os);
 
-  if (conf.mode != MD_USERLAND && conf.mode != MD_SBIN)
+  if (conf.mode == MD_KERNEL || conf.mode == MD_MODULE)
     {
       fprintf (f, "CFLAGS=-D_KERNEL\n");
 #ifdef sun
@@ -764,6 +764,11 @@
 #endif
 
     }
+  else
+    {
+      fprintf (f, "CFLAGS=-O\n");
+    }
+
 #if !defined(__SCO_VERSION__)
   if (*conf.cflags != 0)
     fprintf (f, "CFLAGS += %s\n", conf.cflags);

_______________________________________________
oss-devel mailing list
oss-devel@mailman.opensound.com
http://mailman.opensound.com/mailman/listinfo/oss-devel