[MPlayer-dev-eng] [PATCH] Set HAVE_FAST_CLZ according to CPU type
Diego Biurrun
diego at biurrun.de
Tue Jan 19 11:37:09 CET 2010
On Tue, Jan 19, 2010 at 10:55:44AM +0800, Zuxy Meng wrote:
> 2010/1/18 Diego Biurrun <diego at biurrun.de>:
> > On Mon, Jan 18, 2010 at 03:00:25PM +0800, Zuxy Meng wrote:
> >>
> >> The attached patch adds support for detecting fast CLZ.
> >
> > Fine with me..
> >
> >> --- configure (???? 30363)
> >> +++ configure (????????)
> >> @@ -2761,6 +2781,10 @@
> >>
> >> + if test _armv5te = "yes" && test $_fast_clz = "auto" ; then
> >> + _fast_clz=yes
> >> + fi
> >
> > Could be done on one line with && and && ..
>
> Updated in attached patch.
IMO just apply if you have addressed all comments. One less iteration
and I can still raise concerns on -cvslog.
Diego
More information about the MPlayer-dev-eng
mailing list