[PATCH] D124974: [clang] Include clang config.h in LangStandards.cpp

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 10 11:23:44 PDT 2022


dexonsmith added a comment.

In D124974#3494374 <https://reviews.llvm.org/D124974#3494374>, @sammccall wrote:

> In D124974#3494279 <https://reviews.llvm.org/D124974#3494279>, @dexonsmith wrote:
>
>> Can we add a test for this?
>
> I think the problem with testing it is that this variable comes from the CMake config, and in the default configuration (as used by all the bots) it's not set.

Maybe you could use the same CMake config to set a lit variable, and use that lit variable to test that we get the configured default language variant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124974



More information about the cfe-commits mailing list