[libc-commits] [libc] [libc] Add a developer doc about adding new config options. (PR #66432)

via libc-commits libc-commits at lists.llvm.org
Thu Sep 14 14:06:22 PDT 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc
            
<details>
<summary>Changes</summary>
None
--
Full diff: https://github.com/llvm/llvm-project/pull/66432.diff

2 Files Affected:

- (added) libc/docs/dev/config_options.rst (+138) 
- (modified) libc/docs/dev/index.rst (+1) 


<pre>
diff --git a/libc/docs/dev/config_options.rst b/libc/docs/dev/config_options.rst
new file mode 100644
index 000000000000000..507a5538918c909
--- /dev/null
+++ b/libc/docs/dev/config_options.rst
@@ -0,0 +1,138 @@
+.. _configure_options:
+
+=================================
+Adding new libc configure options
+=================================
+
+`There are a number of configure options &lt;../configure.html&gt;`_ which can be used
+to configure the libc build. The config system is driven by a set of
+hierarchical JSON files. At the top of the hierarchy is a JSON file by name
+``config.json`` in the ``config`` directory. This JSON file lists the libc
+options which affect all platforms. The default value for the option and a short
+description about it listed against each option. For example:
+
+.. code-block::
+
+   {
+     &quot;printf&quot;: {
+       &quot;LIBC_CONF_PRINTF_DISABLE_FLOAT&quot;: {
+         &quot;value&quot;: false,
+         &quot;doc&quot;: &quot;Disable printing floating point values in printf and friends.&quot;
+       },
+       ...
+     }
+   }
+
+The above config indicates that the option ``LIBC_CONF_PRINTF_DISABLE_FLOAT``
+has a value of false. A platform, say the baremetal platform, can choose to
+override this value in its ``config.json`` file in the ``config/baremetal``
+directory with the following contents:
+
+.. code-block::
+
+   {
+     &quot;printf&quot;: {
+       &quot;LIBC_CONF_PRINTF_DISABLE_FLOAT&quot;: {
+         &quot;value&quot;: true
+       }
+     }
+   }
+
+As you can see, the config for the baremetal platform overrides the common
+``false`` value of the ``LIBC_CONF_PRINTF_DISABLE_FLOAT`` with the ``true``
+value.
+
+Config JSON format
+==================
+
+Named tags
+----------
+
+As can be noted from the above examples, ``config.json`` files contains a
+top-level dictionary. The keys of this dictionary are the names of
+*grouping-tags*. A grouping-tag is nothing but a named tag to refer to a related
+group of libc options. In the above example, a tag named ``printf`` is used to
+group all libc options which affect the behavior of ``printf`` and friends.
+
+Tag values
+----------
+
+The value corresponding to each grouping tag is also a dictionary called the
+*option-dictionary*. The keys of the option-dictionary are the names of the libc
+options belonging to that grouping tag. For the ``printf`` tag in the above
+example, the option-dictionary is:
+
+.. code-block::
+
+   {
+     &quot;LIBC_CONF_PRINTF_DISABLE_FLOAT&quot;: {
+       &quot;value&quot;: false,
+       &quot;doc&quot;: 
+     },
+     ...
+   }
+
+The value corresponding to an option key in the option-dictionary is another
+dictionary with two keys: ``&quot;value&quot;`` and ``&quot;doc&quot;``. The ``&quot;value&quot;`` key has
+the value of the option listed against it, and the ``&quot;doc&quot;`` key has a short
+description of the option listed against it. Note that only the main config
+file ``config/config.json`` includes the ``&quot;doc&quot;`` key. Options which are of
+``ON``/``OFF`` kind take boolean values ``true``/``false``. Other types of
+options can take an integral or string value as suitable for that option. In
+the above option-dictionary, the option-key ``LIBC_CONF_PRINTF_DISABLE_FLOAT``
+is of boolean type with value ``true``.
+
+Option name format
+------------------
+
+The option names, or the keys of a option-dictionary, have the following format:
+
+.. code-block::
+
+   LIBC_CONF_&lt;UPPER_CASE_TAG_NAME&gt;_&lt;ACTION_INDICATING_THE_INTENDED_SEMANTICS&gt;
+
+The option name used in the above examples, ``LIBC_CONF_PRINTF_DISABLE_FLOAT``
+to disable printing of floating point numbers, follows this format: It has the
+prefix ``LIBC_CONF_``, followed by the grouping-tag name ``PRINTF`` in upper
+case, followed by the action to disable floating point number printing
+``LIBC_CONF_PRINTF_DISABLE_FLOAT``.
+
+Mechanics of config application
+===============================
+
+Config reading
+--------------
+
+At libc config time, three different ``config.json`` files are read in the
+following order:
+
+1. ``config/config.json``
+2. ``config/&lt;platform or OS&gt;/config.json`` if present.
+3. ``config/&lt;platform or OS&gt;/&lt;target arch&gt;/config.json`` if present.
+
+Each successive ``config.json`` file overrides the option values set by
+previously read ``config.json`` files. Likewise, a similarly named command line
+option to the cmake command will override the option values specified in all or
+any of these ``config.json`` files. That is, users will be able to override the
+config options from the command line.
+
+Config application
+------------------
+
+Local to the directory where an option group is relevant, suitable build logic
+should convert the CMake config options to appropriate compiler and/or linker
+flags. Those compile/link flags can be used in listing the affected targets as
+follows:
+
+.. code-block::
+
+   add_object_library(
+    ...
+    COMPILE_OPTIONS
+      ${common_printf_compile_options}
+      ... # Other compile options affecting this target irrespective of the
+          # libc config options
+   )
+
+Note that the above scheme is only an example and not a prescription.
+Developers should employ a scheme appropriate to the option being added.
diff --git a/libc/docs/dev/index.rst b/libc/docs/dev/index.rst
index 158089929b3af40..87712afcae2ac6f 100644
--- a/libc/docs/dev/index.rst
+++ b/libc/docs/dev/index.rst
@@ -12,6 +12,7 @@ Navigate to the links below for information on the respective topics:
    source_tree_layout
    entrypoints
    cmake_build_rules
+   config_options
    clang_tidy_checks
    fuzzing
    ground_truth_specification
</pre>
</details>


https://github.com/llvm/llvm-project/pull/66432


More information about the libc-commits mailing list