[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