This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Use __glibc_(un)likely instead __builtin_expect.
- From: Rich Felker <dalias at aerifal dot cx>
- To: Roland McGrath <roland at hack dot frob dot com>
- Cc: Siddhesh Poyarekar <siddhesh dot poyarekar at gmail dot com>, OndÅej BÃlka <neleai at seznam dot cz>, GNU C Library <libc-alpha at sourceware dot org>
- Date: Fri, 25 Oct 2013 20:06:25 -0400
- Subject: Re: [PATCH] Use __glibc_(un)likely instead __builtin_expect.
- Authentication-results: sourceware.org; auth=none
- References: <20131022220131 dot GA30971 at domone dot podge> <CAAHN_R2OzxGbDarALVD5rRJP6p7OZYh4di3MaTqcoRnLkaU8JQ at mail dot gmail dot com> <20131025213354 dot F389D746BA at topped-with-meat dot com>
On Fri, Oct 25, 2013 at 02:33:54PM -0700, Roland McGrath wrote:
> > I have not done a full review of the patch, but I am of the opinion
> > that if you can show that the generated code is identical, then such a
> > cleanup patch should be OK. I'd like opinions from others from this
> > though, so please wait for more reviews before you push this change.
>
> If generated code is identical and source changes introduce no style
> violations (missing whitespace, overlong lines), then it's fine.
Is any of this code shared with gnulib? That might be an issue.
Rich