llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Krystian Stasiowski (sdkrystian) <details> <summary>Changes</summary> According to the [Doxygen documentation](https://www.doxygen.nl/manual/commands.html#cmdrelates), the `relates`, `related`, `relatesalso`, and `relatedalso` commands all have a single argument. This patch changes their classification from `VerbatimLineCommand` to `InlineCommand` so the argument is correctly parsed. --- Full diff: https://github.com/llvm/llvm-project/pull/115040.diff 2 Files Affected: - (modified) clang/docs/ReleaseNotes.rst (+2) - (modified) clang/include/clang/AST/CommentCommands.td (+5-5) ``````````diff diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 4e555914caee8a..fda5b0213e7ec7 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -598,6 +598,8 @@ Bug Fixes to AST Handling - Fixed a crash that occurred when dividing by zero in complex integer division. (#GH55390). - Fixed a bug in ``ASTContext::getRawCommentForAnyRedecl()`` where the function could sometimes incorrectly return null even if a comment was present. (#GH108145) +- Clang now correctly parses the argument of the ``relates``, ``related``, ``relatesalso``, + and ``relatedalso`` comment commands. Miscellaneous Bug Fixes ^^^^^^^^^^^^^^^^^^^^^^^ diff --git a/clang/include/clang/AST/CommentCommands.td b/clang/include/clang/AST/CommentCommands.td index a410cd4039bee1..f6dd67dc2590f0 100644 --- a/clang/include/clang/AST/CommentCommands.td +++ b/clang/include/clang/AST/CommentCommands.td @@ -111,6 +111,11 @@ def Extends : InlineCommand<"extends">; def Implements : InlineCommand<"implements">; def MemberOf : InlineCommand<"memberof">; +def Relates : InlineCommand<"relates">; +def Related : InlineCommand<"related">; +def RelatesAlso : InlineCommand<"relatesalso">; +def RelatedAlso : InlineCommand<"relatedalso">; + //===----------------------------------------------------------------------===// // BlockCommand //===----------------------------------------------------------------------===// @@ -248,11 +253,6 @@ def Page : VerbatimLineCommand<"page">; def Mainpage : VerbatimLineCommand<"mainpage">; def Subpage : VerbatimLineCommand<"subpage">; -def Relates : VerbatimLineCommand<"relates">; -def Related : VerbatimLineCommand<"related">; -def RelatesAlso : VerbatimLineCommand<"relatesalso">; -def RelatedAlso : VerbatimLineCommand<"relatedalso">; - def AddIndex : VerbatimLineCommand<"addindex">; // These take a single argument mostly, but since they include a file they'll `````````` </details> https://github.com/llvm/llvm-project/pull/115040 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
