<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div style="FONT-SIZE: 14px; COLOR: #000000; LINE-HEIGHT: 1.7; FONT-FAMILY: arial"><pre><br>At 2015-04-10 23:30:48,"Steve Borho" <<a href="mailto:steve@borho.org">steve@borho.org</a>> wrote:
>On 04/10, Min Chen wrote:
>> # HG changeset patch
>> # User Min Chen <<a href="mailto:chenm003@163.com">chenm003@163.com</a>>
>> # Date 1428670165 -28800
>> # Node ID 43792061e4f115b5257caf3ca8cd95cf561ec888
>> # Parent  67f55ea84d7b6e4f872d5bddf7aa8dec971fef60
>> avoid calculate rateIncUp and rateIncDown when sigHide disabled
>> ---
>>  source/common/quant.cpp |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>> 
>> diff -r 67f55ea84d7b -r 43792061e4f1 source/common/quant.cpp
>> --- a/source/common/quant.cpp        Fri Apr 10 20:06:28 2015 +0800
>> +++ b/source/common/quant.cpp        Fri Apr 10 20:49:25 2015 +0800
>> @@ -739,7 +739,7 @@
>>                  totalRdCost += costCoeff[scanPos];
>>  
>>                  /* record costs for sign-hiding performed at the end */
>> -                if (level)
>> +                if ((cu.m_slice->m_pps->bSignHideEnabled ? ~0 : 0) & level)
>
>this is faster than if (level && cu.m_slice->m_pps->bSignHideEnabled) ?
yes, mask operator just generate one conditional jump<br>of course,  we need make a new variant for mask on some compiler to avoid duplicate mask generate.</pre></div></div>