[cfe-commits] r148873 - in /cfe/trunk: include/clang/Basic/Builtins.def lib/Sema/SemaDecl.cpp test/SemaObjC/builtin_objc_nslog.m

[Jean-Daniel Dupas]

Le 25 janv. 2012 à 01:45, NAKAMURA Takumi a écrit :

> 2012/1/25 Jean-Daniel Dupas <devlists at shadowlab.org>:
>> Author: jddupas
>> Date: Tue Jan 24 16:32:46 2012
>> New Revision: 148873
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=148873&view=rev
>> Log:
>> Replace a hack to handle NSLog/NSLogv in sema by declaring them as Library Builtins.
> 
> Jean-Daniel,
> 
> I saw an unexpected diagnostic on msvc and cygwin.
> 
> error: 'warning' diagnostics expected but not seen:
>  Line 11: implicitly declaring C library function 'NSLogv' with type
> 'void (id, __va_list_tag *)'
> error: 'warning' diagnostics seen but not expected:
>  Line 11: implicitly declaring C library function 'NSLogv' with type
> 'void (id, __builtin_va_list)'
> 2 errors generated.
> 
> I have confirmed it would be fixed if "-triple x86_64-darwin" were
> added. I wonder which triplet would be more suitable.
> 
> ...Takumi


The va_list part is not require in this test, so we can probably simply remove it to workaround this issue.


-- Jean-Daniel



-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.patch
Type: application/octet-stream
Size: 648 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120125/506895e8/attachment.obj>