This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v2] Add math-inline benchmark


On 07/23/2015 11:39 AM, OndÅej BÃlka wrote:
>> No that wouldn't make any difference - all that matters for that experiment was 
>> encapsulating the immediate in a separate function, you'll get one call every 
>> iteration either way.
>>
> Thats completely false as my implementation shows that different
> implementation is faster than yours when I use noinline with specific
> expressions. For example these could be invalid as gcc decides to turn
> all implementations into branchless code which completely changes
> performance characteristic.

When a conversation gets to a sufficient length it is beneficial for one party
in the conversation to summarize the current position without responding point
by point through a long thread.

It would be immensely beneficial to additional reviewers to start the thread
again from the v3 version of the patch, and instead of responding point by point,
respond at a high level with a summary of what has been agreed upon and what
issues are still outstanding.

With a sufficiently complex topic we are going to need to learn how to have
proper discussions that gravitate towards consensus rather than spiral apart
towards long threads. That is to say that the emails should get shorter and
shorter as fewer and fewer topics are left to clarify and discuss after
consensus is reached one-by-one on each point.

Cheers,
Carlos.
 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]