[tremor] [patch] configuring Tremor natively on ARM misses ARM optimizations

Nicolas Pitre nico at cam.org
Fri Nov 28 17:50:04 PST 2003



Monty,

Could you please commit this patch to both HEAD and the low-mem branch:

Index: configure.in
===================================================================
RCS file: /usr/local/cvsroot/Tremor/configure.in,v
retrieving revision 1.4
diff -u -r1.4 configure.in
--- configure.in	16 Oct 2002 09:07:00 -0000	1.4
+++ configure.in	29 Nov 2003 01:27:48 -0000
@@ -41,7 +41,7 @@
 ldflags_save="$LDFLAGS"
 if test -z "$GCC"; then
         case $host in 
-        arm-*-*)
+        arm*-*-*)
                 DEBUG="-g -D_ARM_ASSEM_" 
                 CFLAGS="-O -D_ARM_ASSEM_"
                 PROFILE="-p -g -O -D_ARM_ASSEM" ;;
@@ -53,7 +53,7 @@
 else
 
         case $host in 
-        arm-*-*)
+        arm*-*-*)
                 DEBUG="-g -W -D__NO_MATH_INLINES -fsigned-char -D_ARM_ASSEM_"
                 CFLAGS="-O2 -D_ARM_ASSEM_ -fsigned-char"
                 PROFILE="-W -pg -g -O2 -D_ARM_ASSEM_ -fsigned-char -fno-inline-functions";;

The canonical name on a native ARM system is either armv3-*-*,
armv4-*-*, etc.  Without this, configuring Tremor natively is missing 
the ARM specific optimizations entirely.

<p>Nicolas

--- >8 ----
List archives:  http://www.xiph.org/archives/
Ogg project homepage: http://www.xiph.org/ogg/
To unsubscribe from this list, send a message to 'tremor-request at xiph.org'
containing only the word 'unsubscribe' in the body.  No subject is needed.
Unsubscribe messages sent to the list will be ignored/filtered.



More information about the Tremor mailing list