[MPlayer-dev-eng] [PATCH] Enable FTZ/DAZ for SSE

[Zuxy Meng]

Hi,

2007/5/11, Zuxy Meng <zuxy.meng at gmail.com>:
> Hi,
>
> 2007/3/30, Zuxy Meng <zuxy.meng at gmail.com>:
> > Hi,
> >
> > 2007/3/20, Zuxy Meng <zuxy.meng at gmail.com>:
> > > Hi,
> > >
> > > 2007/3/7, Loren Merritt <lorenm at u.washington.edu>:
> > > > On Tue, 6 Mar 2007, Zuxy Meng wrote:
> > > > > 2007/3/6, Diego Biurrun <diego at biurrun.de>:
> > > > >> On Tue, Mar 06, 2007 at 07:46:47PM +0800, Zuxy Meng wrote:
> > > > >>>
> > > > >>>  mplayer$(EXESUF): $(MPLAYER_DEPS)
> > > > >>> -     $(CC) -o $@ $^ $(LDFLAGS_MPLAYER)
> > > > >>> +     $(CC) $(CFLAGS) -o $@ $^ $(LDFLAGS_MPLAYER)
> > > > >>
> > > > >> Looks incorrect.  This is the linking stage, compilation flags should
> > > > >> not be necessary here.
> > > > >
> > > > > There's an "endfile" line in the specs file which looks like
> > > > > %{ffast-math|funsafe-math-optimizations:crtfastmath.o%s}
> > > > >
> > > > > So an additional object file gets linked in if -ffast-math or
> > > > > -funsafe-math-optimizations is in the command line. It doesn't affect
> > > > > compilation really but the link stage instead.
> > > >
> > > > Then those flags should go in LDFLAGS in addition to CFLAGS.
> > >
> > > But actually this flag isn't passed to a linker; only gcc can recognize it.
> >
> > More comments on this?
>
> Seems nobody's interested:-( Anyway I think the patch didn't hurt
> anything, although it would benefit only corner cases.

If no more objections I plan to apply the attached patch next week.
-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6
-------------- next part --------------
Index: Makefile
===================================================================
--- Makefile	?????? 23339??
+++ Makefile	????????????
@@ -223,10 +223,10 @@
 	$(MAKE) -C libmenu
 
 mplayer$(EXESUF): $(MPLAYER_DEPS)
-	$(CC) -o $@ $^ $(LDFLAGS_MPLAYER)
+	$(CC) $(CFLAGS) -o $@ $^ $(LDFLAGS_MPLAYER)
 
 mencoder$(EXESUF): $(MENCODER_DEPS)
-	$(CC) -o $@ $^ $(LDFLAGS_MENCODER)
+	$(CC) $(CFLAGS) -o $@ $^ $(LDFLAGS_MENCODER)
 
 codec-cfg$(EXESUF): codec-cfg.c codec-cfg.h help_mp.h
 	$(HOST_CC) -I. -DCODECS2HTML $< -o $@