[patch] Fix pr9701

Eli Friedman eli.friedman at gmail.com
Mon Sep 23 15:23:10 PDT 2013


On Mon, Sep 23, 2013 at 2:17 PM, Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:

> On 5 September 2013 14:27, Rafael EspĂ­ndola <rafael.espindola at gmail.com>
> wrote:
> > On 3 September 2013 21:11, Richard Smith <richard at metafoo.co.uk> wrote:
> >> What about -munknown-to-clang-option? We really want an error on those
> ones,
> >> since they can change the ABI.
> >
> > The behavior of -funknown-to-clang-option, -Wunknown-to-clang-option
> > and -munknown-to-clang-option is unchanged (we just warn). I think we
> > should be more strict about these too, but since they are the most
> > common options I would like to change them in a follow up patch.
>
> Ping.
>
>
Please add tests for the current behavior anyway, so it's clear what the
patch does and does not do.

Otherwise LGTM.

-Eli
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130923/30ec5fa7/attachment.html>


More information about the cfe-commits mailing list