[PATCH] Fixing warnings revealed by gcc release build

David Blaikie dblaikie at gmail.com
Wed Jan 30 10:52:56 PST 2013


On Wed, Jan 30, 2013 at 8:24 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Wed, Jan 30, 2013 at 6:22 PM, Nico Weber <thakis at chromium.org> wrote:
>> On Wed, Jan 30, 2013 at 7:29 AM, Vane, Edwin <edwin.vane at intel.com> wrote:
>>> Dmitri wrote:
>>>> If this is the only false positive, I think it is worth working around the warning.
>>>
>>> This is indeed the only instance of this empty {} warning bug in the gcc build.
>>
>> There will likely be more in the future.
>>
>> The clang bots should catch the problems detected by this warning
>> reliably without false positives, right?
>
> Not now.  This warning is off by default in Clang:
>
> def warn_missing_field_initializers : Warning<
>   "missing field '%0' initializer">,
>   InGroup<MissingFieldInitializers>, DefaultIgnore;

Then we should probably turn it on (for the LLVM builds) if it's useful.



More information about the cfe-commits mailing list