[Lldb-commits] [lldb] [lldb][Docs] Document our major differences from the LLVM style (PR #66345)
Jonas Devlieghere via lldb-commits
lldb-commits at lists.llvm.org
Thu Sep 14 09:57:11 PDT 2023
================
@@ -18,19 +18,38 @@ Please refer to the `LLVM Developer Policy
authoring and uploading a patch. LLDB differs from the LLVM Developer
Policy in the following respects.
- - **Test infrastructure**: Like LLVM it is important to submit tests with your
- patches, but note that LLDB uses a different system for tests. Refer to the
- `test documentation <test.html>`_ for more details and the ``lldb/test``
- folder on disk for examples.
-
- - **Coding Style**: LLDB's code style differs from
- `LLVM's coding style <https://llvm.org/docs/CodingStandards.html>`_.
- Unfortunately there is no document describing the differences. Please be
- consistent with the existing code.
-
For anything not explicitly listed here, assume that LLDB follows the LLVM
policy.
+Coding Style
+++++++++++++
+
+LLDB's code style differs from `LLVM's coding style <https://llvm.org/docs/CodingStandards.html>`_
+in a few ways. The 2 main ones are:
+
+* `Variable naming <https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly>`_:
+ LLDB prefers variables to be ``named_like_this`` and uses the ``m_`` prefix for
+ member variables.
+
+* `Use of asserts <https://llvm.org/docs/CodingStandards.html#assert-liberally>`_:
+ See the :ref:`section below<Error Handling>`.
----------------
JDevlieghere wrote:
I don't actually think our way of using asserts is all that different from how they are (should be?) used in LLVM. I think it's fair to call it out here as we have extensive documentation below.
https://github.com/llvm/llvm-project/pull/66345
More information about the lldb-commits
mailing list