[cfe-commits] Add -Wno-extension-used to filter ext_token_used

Eli Friedman eli.friedman at gmail.com
Tue Oct 4 15:24:59 PDT 2011


On Tue, Oct 4, 2011 at 11:13 AM, Ivan Krasin <krasin at chromium.org> wrote:
> Hi llvm team!
>
> This patch adds support of -Wno-extension-used to make it possible to
> suppress the warning:
>
> lala.c:1:21: warning: extension used [-pedantic]
> double lala(double) asm("llvm.log.f64");
>
> r140770 has started to promote this warning to error if -Werror is
> specified (which is absolutely correct).
> We have a test that checks some llvm intrinsics and it's not Clangy to
> drop -Werror just because one particular warning is expected.
> I have added the option to suppress this warning and has removed it
> from the test for warnings w/o suppression options.
>
> OK to commit?

We really ought to clarify what exactly this is warning about and name
the warning group in a more intuitive way.  -Wno-extension-used
doesn't give the user any idea what warning it is actually
suppressing.  I'm drawing a blank as to what exactly to call it,
though.

-Eli



More information about the cfe-commits mailing list