blob: 6bad6a656115c6118704c19f54504149d06aed5c [file] [log] [blame]
Marek Kurdejd5bc4d92020-12-02 08:52:35 +01001.. _ContributingToLibcxx:
2
3======================
4Contributing to libc++
5======================
6
7.. contents::
8 :local:
9
10Please read `this document <https://www.llvm.org/docs/Contributing.html>`__ on general rules to contribute to LLVM projects.
11
12Tasks and processes
13===================
14
15This file contains notes about various tasks and processes specific to libc++.
16
Louis Dionne31781652021-01-08 17:23:16 -050017Looking for pre-existing reviews
18================================
19
20Before you start working on any feature, please take a look at the open reviews
21to avoid duplicating someone else's work. You can do that by going to the website
22where code reviews are held, `Differential <https://reviews.llvm.org/differential>`__,
23and clicking on ``Libc++ Open Reviews`` in the sidebar to the left. If you see
24that your feature is already being worked on, please consider chiming in instead
25of duplicating work!
26
Marek Kurdejd5bc4d92020-12-02 08:52:35 +010027Post-Release TODO
28=================
29
30After branching for an LLVM release:
31
321. Update ``_LIBCPP_VERSION`` in ``include/__config``
332. Update the ``include/__libcpp_version`` file
343. Update the version number in ``docs/conf.py``
35
Mark de Wever2bf75c02021-03-24 19:54:40 +010036Modifying feature-test macros
Marek Kurdejf14302f2020-12-10 08:37:21 +010037=============================
38
Mark de Wever2bf75c02021-03-24 19:54:40 +010039When adding or updating feature-test macros, you should update the corresponding tests.
40To do that, modify ``feature_test_macros`` table in the script
41``utils/generate_feature_test_macro_components.py``, run it, and commit updated
42files. Running ``utils/generate_feature_test_macro_components.py`` should never
43generate diffs in a clean checkout; feel free to run it in your local checkout
44any time you want.
45
Marek Kurdejf14302f2020-12-10 08:37:21 +010046
Marek Kurdejd5bc4d92020-12-02 08:52:35 +010047Adding a new header TODO
48========================
49
50When adding a new header to libc++:
51
521. Add a test under ``test/libcxx`` that the new header defines ``_LIBCPP_VERSION``. See ``test/libcxx/algorithms/version.pass.cpp`` for an example.
Arthur O'Dwyer7deec122021-03-24 18:19:12 -0400532. Run ``python utils/generate_header_tests.py``; verify and commit the changes.
543. Modify ``python utils/generate_header_inclusion_tests.py``; run it; verify and commit the changes.
554. Create a submodule in ``include/module.modulemap`` for the new header.
565. Update the ``include/CMakeLists.txt`` file to include the new header.
Marek Kurdejd5bc4d92020-12-02 08:52:35 +010057
58Exporting new symbols from the library
59======================================
60
Mark de Wever2bf75c02021-03-24 19:54:40 +010061When exporting new symbols from libc++, you must update the ABI lists located in ``lib/abi``.
Marek Kurdejd5bc4d92020-12-02 08:52:35 +010062To test whether the lists are up-to-date, please run the target ``check-cxx-abilist``.
63To regenerate the lists, use the target ``generate-cxx-abilist``.
Mark de Wever2bf75c02021-03-24 19:54:40 +010064The ABI lists must be updated for all supported platforms; currently Linux and
65Apple. If you don't have access to one of these platforms, you can download an
66updated list from the failed build at
67`Buildkite <https://buildkite.com/llvm-project/libcxx-ci>`__.
68Look for the failed build and select the ``artifacts`` tab. There, download the
69abilist for the platform, e.g.:
70
71* C++20 for the Linux platform.
72* MacOS C++20 for the Apple platform.