On 4 August 2014 17:25, Reid Kleckner <[email protected]> wrote:
> Some of these flags are a bit functional, like -flto, -fwhole-program, and 
> -fuse-linker-plugin.

-flto is used, what the patch adds is -flto= which is not.

> Rafael, do you have any thoughts about this list, since you added the "error 
> on unknown options" logic?

I added it mostly because clang was warning on typos, which was really
annoying. I agree that  a specific warning is the best default for gcc
options that we know exist but we don't use.

This change LGTM, but should probably get an OK from another clang dev too.

Cheers,
Rafael
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to