[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