[PATCH] D55150: Emit warnings from the driver for use of -mllvm or -Xclang options.

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 6 11:41:49 PST 2018


efriedma added a comment.

I'm not sure that putting a warning that can be disabled really helps here; anyone who needs the option will just disable the warning anyway, and then users adding additional options somewhere else in the build system will miss the warning.

Instead, it would probably be better to rename Xclang and mllvm to something that makes it clear the user is doing something unsupported. Maybe "--unstable-llvm-option" and "--unstable-clang-option" or something like that.  (This will lead to some breakage, but the breakage is roughly equivalent for anyone using -Werror.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55150/new/

https://reviews.llvm.org/D55150





More information about the cfe-commits mailing list