[MPlayer-dev-eng] [PATCH] Set HAVE_FAST_CLZ according to CPU type

Zuxy Meng zuxy.meng at gmail.com
Tue Jan 19 14:35:27 CET 2010


2010/1/19 Diego Biurrun <diego at biurrun.de>:
> 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.

OK. Thanks! Applied.

-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6



More information about the MPlayer-dev-eng mailing list