[PATCH] D52773: clang-cl: Add /showFilenames option (PR31957)

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 4 10:38:12 PDT 2018


thakis added a comment.

In https://reviews.llvm.org/D52773#1255093, @zturner wrote:

> I agree magic environment variables are bad, but without it we don’t
>  address the only current actual use we have for this, which is making the
>  vs integration print filenames. Detecting compiler version from inside the
>  integration is hard


We need some solution to this anyhow; e.g. say "this now requires clang 8.0", or have a clang version dropdown in the UI (which defaults to the latest release), or something. We can't add an env var for every future flag that the vs integration might want to use.


https://reviews.llvm.org/D52773





More information about the cfe-commits mailing list