Re: [PATCH] fix size_estimation for builtin_expect

2013-10-03 Thread Jan Hubicka
> attached the new patch. OK for check in? OK, thanks! Honza

Re: [PATCH] fix size_estimation for builtin_expect

2013-10-02 Thread Rong Xu
attached the new patch. OK for check in? On Wed, Oct 2, 2013 at 9:12 AM, Jan Hubicka wrote: >> > Hi, >> > >> > builtin_expect should be a NOP in size_estimation. Indeed, the call >> > stmt itself is 0 weight in size and time. But it may introduce >> > an extra relation expr which has non-zero siz

Re: [PATCH] fix size_estimation for builtin_expect

2013-10-02 Thread Jan Hubicka
> > Hi, > > > > builtin_expect should be a NOP in size_estimation. Indeed, the call > > stmt itself is 0 weight in size and time. But it may introduce > > an extra relation expr which has non-zero size/time. The end result > > is: for w/ and w/o builtin_expect, we have different size/time estimatio

Re: [PATCH] fix size_estimation for builtin_expect

2013-10-01 Thread Rong Xu
ping. On Fri, Sep 27, 2013 at 3:56 PM, Rong Xu wrote: > Hi, > > builtin_expect should be a NOP in size_estimation. Indeed, the call > stmt itself is 0 weight in size and time. But it may introduce > an extra relation expr which has non-zero size/time. The end result > is: for w/ and w/o builtin_e