[cfe-dev] Patch for __builtin_bzero

[Nico Weber]

On 19.07.2008, at 20:24, Eli Friedman wrote:

> On Sat, Jul 19, 2008 at 12:04 AM, Nico Weber <nicolasweber at gmx.de>  
> wrote:
>> Hi,
>>
>> with the attached patch, vim7 can be built with `make CC=ccc`.  
>> Linking takes
>> about 3 minutes (with a debug build of llvm), and vim's build  
>> process runs
>> the linker several times (at least 3 times), so total linking time  
>> is about
>> 9 minutes.
>
> Okay, cool... although the difference between debug and release is
> pretty big.  clang is roughly 2x faster in release mode, and linking
> is roughly 5x faster, if I recall correctly.
>
>> __builtin_bzero() simply calls the library's bzero() in this patch.
>> Would it be better to use the llvm.memset intrinsic instead? I  
>> believe
>> Windows does not have bzero in its c library.
>
> I'd say don't bother for the moment; if it becomes an issue, we can
> deal with it later.


Great. Can someone commit the patch? I changed the test so that it  
does no longer compile and execute the code, it now uses grep.

Nico

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzero.patch
Type: application/octet-stream
Size: 489 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20080721/78552ce6/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: builtin-bzero.c
Type: application/octet-stream
Size: 140 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20080721/78552ce6/attachment-0001.obj>