r258128 - Add -Wexpansion-to-undefined: warn when using `defined` in a macro definition.

[Diego Novillo via cfe-commits]

Thanks!
On Jan 19, 2016 3:34 PM, "Nico Weber" <thakis at google.com> wrote:

> I think I've fixed all warnings by now, and I've reenabled the warning for
> LLVM builds (in r258210; that commit's commit message links to the
> individual fixes.)
>
> On Tue, Jan 19, 2016 at 3:56 PM, Nico Weber <thakis at google.com> wrote:
>
>> r258181 should stop the bleeding. I'll look at fixing the warnings now.
>>
>> On Tue, Jan 19, 2016 at 3:46 PM, Nico Weber <thakis at google.com> wrote:
>>
>>> Back at my desk now, looking.
>>>
>>> On Tue, Jan 19, 2016 at 3:44 PM, Diego Novillo <dnovillo at google.com>
>>> wrote:
>>>
>>>>
>>>>
>>>> On Tue, Jan 19, 2016 at 3:43 PM, Nico Weber <thakis at google.com> wrote:
>>>>
>>>>> I mean you could pass a -Wno flag. It's undefined behavior that's
>>>>> actually causing bugs in practice; it should probably be on by default.
>>>>>
>>>>
>>>> But then you need to fix all the warnings it's producing in an llvm
>>>> build.  That is currently blocking several folks (us included).
>>>>
>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160119/daff26d1/attachment-0001.html>