[libcxx-commits] [PATCH] D81846: [libc++] Allow specifying custom Lit config files

[Louis Dionne via Phabricator via libcxx-commits]

ldionne created this revision.
ldionne added a reviewer: gargaroff.
Herald added subscribers: libcxx-commits, dexonsmith, jkorous, mgorny.
Herald added a project: libc++.
Herald added a reviewer: libc++.
ldionne added reviewers: CaseyCarter, STL_MSFT, jwakely.
ldionne added a comment.

Adding reviewers whom I think might be impacted by this change.


Before this patch, the libc++ test suite first loads lit.site.cfg
(generated by CMake), and then lit.cfg. It's also possible to load
lit.cfg before lit.site.cfg and to point to a custom lit.site.cfg
file using '--param=libcxx_site_config'. However, in that case, lit.cfg
still relies on the site configuration filling up the 'config' object
like the default lit.site.cfg file does, which isn't flexible enough.

This commit simplifies the setup by having just a single Lit site config
file per CMake configuration, and always loading exactly that config file.
However, the config file to use can be selected when setting up CMake via
the LIBCXX_TEST_CONFIG setting. Furthermore, the site configs are entirely
standalone, which means that a new site config can be added that doesn't
need to conform what's expected by config.py.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D81846

Files:
  libcxx/CMakeLists.txt
  libcxx/docs/TestingLibcxx.rst
  libcxx/test/CMakeLists.txt
  libcxx/test/lit.cfg
  libcxx/test/lit.cfg.py
  libcxx/test/lit.site.cfg.in

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81846.270768.patch
Type: text/x-patch
Size: 9182 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20200615/6c3ec3aa/attachment-0001.bin>