Re: Official request: Please make GNU grep the default

From: Adrian Chadd <adrian_at_freebsd.org>
Date: Fri, 20 Aug 2010 20:36:26 +0800
2010/8/20 Dag-Erling Smørgrav <des_at_des.no>:
> Adrian Chadd <adrian_at_freebsd.org> writes:
>> I've just looked at grep_fgetln(). Surely memchr() isn't required there.
>
> Of course it is, how else are you going to locate the '\n'?  OTOH, I'm
> not sure grep_fgetln() is needed at all.

It seems a bit strange that memchr(), which should be hitting data
that's in the cache (as it was recently read, right?) is showing up so
high in the profiling results. memchr() in libc/string/memchr.c looks
like how I'd inline it, so, hm.

Have you tried this in pmc?


Adrian
Received on Fri Aug 20 2010 - 10:36:29 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:06 UTC