On 12/11/2015 09:18 PM, Jakub Jelinek wrote:

Unfortunately, my patch broke some cases with warning aliases that happened
to work (by accident) and left some other warning alias cases broken.

This patch attempts to fix that (and add Warning keyword to two warning
aliases that didn't have it), so that -Werror= works even for them again.
As we do nothing beyond cancelling -Werror for -Wno-error=, there is no need
to deal with neg_alias_arg, just alias_arg is enough.

Took me a while to figure out what you were saying with that last sentence, but in the end I agree.

2015-12-11  Jakub Jelinek  <ja...@redhat.com>

        PR c/68833
        * common.opt (Wmissing-noreturn): Add Warning option.
        * opts-common.c (control_warning_option): If opt is
        alias_target with alias_arg, set arg to it.

        * c.opt (Wmissing-format-attribute, Wnormalized): Add Warning option.

        * c-c++-common/pr68833-1.c: New test.
        * c-c++-common/pr68833-2.c: New test.

Ok.


Bernd

Reply via email to