[llvm-commits] Enable -Wcovered-switch-default in our builds

Daniel Dunbar daniel at zuster.org
Tue Feb 28 07:26:31 PST 2012


Hi Rafael,

It turns out that your configure check doesn't work as intended with
Clang, because Clang doesn't error out on unknown warning options.

This makes the build have a lot of warning spew when building with an
older version of Clang.

Can you see about fixing CXX_FLAG_CHECK to handle this case?

 - Daniel

2012/2/27 Rafael Espíndola <rafael.espindola at gmail.com>:
> While  -Wcovered-switch-default was found to be too noisy to most
> projects, it matches our coding style. The attached patch enables it
> in our builds. Is it OK?
>
> Cheers,
> Rafael
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>




More information about the llvm-commits mailing list