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

[Nico Weber via cfe-commits]

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/ba14ae51/attachment.html>