This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Add LIBCXX_EXTRA_SITE_DEFINES for adding extra defines in __config_site
ClosedPublic

Authored by mstorsjo on Dec 21 2021, 7:15 AM.

Details

Summary

This is similar to the existing setting LIBCXX_ABI_DEFINES, with
the difference that this also allows setting other defines than
ones that start with "_LIBCPP_ABI_".

This allows avoiding using LIBCXX_TEST_COMPILER_FLAGS in two
CI configurations.

(Alternatively, we could loosen the requirement on LIBCXX_ABI_DEFINES
about what defines it can set.)

Diff Detail

Event Timeline

mstorsjo created this revision.Dec 21 2021, 7:15 AM
mstorsjo requested review of this revision.Dec 21 2021, 7:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 21 2021, 7:15 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Dec 21 2021, 2:23 PM
This revision is now accepted and ready to land.Dec 21 2021, 2:23 PM