[llvm-branch-commits] [clang] [analyzer][docs] Mention that 'expand-macros' won't use clang-format in the future (PR #187683)

[via llvm-branch-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Balázs Benics (steakhal)

<details>
<summary>Changes</summary>

See the discussion at:
https://discourse.llvm.org/t/can-we-link-clang-format-into-clanganalysis/89014/7

The clang-format macros expansion formatting was reverted from llvm/main in PR #<!-- -->186614.

---
Full diff: https://github.com/llvm/llvm-project/pull/187683.diff


1 Files Affected:

- (modified) clang/docs/ReleaseNotes.rst (+21-2) 


``````````diff
diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index d4c6025ee0638..04d3bc83072c1 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -920,9 +920,28 @@ Crash and bug fixes
 Improvements
 ^^^^^^^^^^^^
 
-- The `expand-macros <https://clang.llvm.org/docs/analyzer/user-docs/Options.html#expand-macros>`__
+.. |ss| raw:: html
+
+   <strike>
+
+.. |se| raw:: html
+
+   </strike>
+
+.. |br| raw:: html
+
+   <br />
+
+- |ss| The `expand-macros <https://clang.llvm.org/docs/analyzer/user-docs/Options.html#expand-macros>`__
   analyzer config option now formats the macro expansions using LLVM-style
-  clang-format. (#GH154743)
+  clang-format. (#GH154743) |se| |br|
+  **EDIT:** This feature was later reverted from upstream LLVM (#GH186614), and
+  it's currently not planned to land again due to the library dependencies it
+  would need. We didn't revert this change from the release branch for ABI
+  compatibility reasons. We are sorry for the inconvinience.
+  Read more about this in the relevant
+  `LLVM Discourse thread <https://discourse.llvm.org/t/can-we-link-clang-format-into-clanganalysis/89014/7>`__.
+
 - ``[[clang::suppress]]`` now can suppress diagnostics within primary templates.
   (#GH168954)
 - Improved the false-positive suppression for ``std::unique_ptr`` and

``````````

</details>


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