[cfe-commits] r86129 - in /cfe/trunk: lib/Sema/Sema.h lib/Sema/SemaCXXCast.cpp lib/Sema/SemaDeclCXX.cpp lib/Sema/SemaOverload.cpp test/SemaTemplate/instantiate-complete.cpp utils/C++Tests/LLVM-Syntax/lit.local.cfg

[Douglas Gregor]

On Nov 5, 2009, at 10:29 AM, Chris Lattner wrote:

>
> On Nov 5, 2009, at 10:29 AM, Douglas Gregor wrote:
>
>>
>> On Nov 5, 2009, at 10:15 AM, Chris Lattner wrote:
>>
>>>
>>> On Nov 5, 2009, at 5:06 AM, Douglas Gregor wrote:
>>>
>>>> +++ cfe/trunk/utils/C++Tests/LLVM-Syntax/lit.local.cfg Thu Nov  5  
>>>> 07:06:35 2009
>>>> @@ -16,6 +16,7 @@
>>>>                                               pattern='^(.*\\.h| 
>>>> [^.]*)$',
>>>>                                               extra_cxx_args=['- 
>>>> D__STDC_LIMIT_MACROS',
>>>>                                                               '- 
>>>> D__STDC_CONSTANT_MACROS',
>>>> +                                                                  
>>>> '-Wno-sign-compare',
>>>>                                                               '-I 
>>>> %s/include' % root.llvm_src_root,
>>>>                                                               '-I 
>>>> %s/include' % root.llvm_obj_root])
>>>>
>>>
>>> Random question: why is this building with -Wno-sign-compare?
>>
>> Because I didn't want to fix all of the places in LLVM's headers  
>> where Clang was spitting out sign-comparison warnings.
>
> We build with GCC with this warning though, it clang producing it in  
> cases where GCC isn't?

Yes. All of the cases where Clang is producing a warning (but GCC  
doesn't) seem to have a cast on at least one side of the == or !=.

	- Doug