ok for google branch.
David
On Fri, Oct 11, 2013 at 3:17 PM, Rong Xu wrote:
> here is the new patch. Note that the hitrate won't change by this
> patch for the case of
> while (__builtin_expect (exp, 0))
>
> Index: predict.c
> ===
here is the new patch. Note that the hitrate won't change by this
patch for the case of
while (__builtin_expect (exp, 0))
Index: predict.c
===
--- predict.c (revision 203462)
+++ predict.c (working copy)
@@ -1951,11 +1951,42 @@
ok. that makes sense.
On Fri, Oct 11, 2013 at 2:41 PM, Xinliang David Li wrote:
> Should it be max + some_delta then?
>
> David
>
> On Fri, Oct 11, 2013 at 2:32 PM, Rong Xu wrote:
>> I want to differentiate the cases w/o and w/ builtin.
>> If I take the max, they will be the same (91%).
>>
>> -
Should it be max + some_delta then?
David
On Fri, Oct 11, 2013 at 2:32 PM, Rong Xu wrote:
> I want to differentiate the cases w/o and w/ builtin.
> If I take the max, they will be the same (91%).
>
> -Rong
>
> On Fri, Oct 11, 2013 at 2:26 PM, Xinliang David Li wrote:
>> Why this 'percent += 4'
I want to differentiate the cases w/o and w/ builtin.
If I take the max, they will be the same (91%).
-Rong
On Fri, Oct 11, 2013 at 2:26 PM, Xinliang David Li wrote:
> Why this 'percent += 4' instead of taking the max?
>
> David
>
> On Fri, Oct 11, 2013 at 2:10 PM, Rong Xu wrote:
>> The trunk
Why this 'percent += 4' instead of taking the max?
David
On Fri, Oct 11, 2013 at 2:10 PM, Rong Xu wrote:
> The trunk version of this patch is submitted for review.
>
> David: can we have this patch for google/gcc-4_8 branch first?
> It tested with regression and google internal benchmarks.
>
>
