Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[clang-format] Fixed the warning in building document for TableGenBreakingDAGArgOperators. #85760

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

hnakamura5
Copy link
Contributor

Intend to fix the Test documentation build , degraded here #83149 .

@llvmbot llvmbot added clang Clang issues not falling into any other category clang-format labels Mar 19, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Mar 19, 2024

@llvm/pr-subscribers-clang

@llvm/pr-subscribers-clang-format

Author: Hirofumi Nakamura (hnakamura5)

Changes

Intend to fix the Test documentation build , degraded here #83149 .


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

2 Files Affected:

  • (modified) clang/docs/ClangFormatStyleOptions.rst (+1-1)
  • (modified) clang/include/clang/Format/Format.h (+1-1)
diff --git a/clang/docs/ClangFormatStyleOptions.rst b/clang/docs/ClangFormatStyleOptions.rst
index 35b6d0a2b52b67..be021dfc5c084c 100644
--- a/clang/docs/ClangFormatStyleOptions.rst
+++ b/clang/docs/ClangFormatStyleOptions.rst
@@ -6204,7 +6204,7 @@ the configuration (without a prefix: ``Auto``).
 
   For example the configuration,
 
-  .. code-block:: c++
+  .. code-block:: yaml
 
     TableGenBreakInsideDAGArg: BreakAll
     TableGenBreakingDAGArgOperators: ['ins', 'outs']
diff --git a/clang/include/clang/Format/Format.h b/clang/include/clang/Format/Format.h
index 54861a66889e22..7ad2579bf7773b 100644
--- a/clang/include/clang/Format/Format.h
+++ b/clang/include/clang/Format/Format.h
@@ -4735,7 +4735,7 @@ struct FormatStyle {
   /// the specified identifiers.
   ///
   /// For example the configuration,
-  /// \code
+  /// \code{.yaml}
   ///   TableGenBreakInsideDAGArg: BreakAll
   ///   TableGenBreakingDAGArgOperators: ['ins', 'outs']
   /// \endcode

@HazardyKnusperkeks HazardyKnusperkeks merged commit 29cde04 into llvm:main Mar 19, 2024
7 of 8 checks passed
@hnakamura5
Copy link
Contributor Author

Thank you so much for quick response!

chencha3 pushed a commit to chencha3/llvm-project that referenced this pull request Mar 23, 2024
…akingDAGArgOperators. (llvm#85760)

Intend to fix the `Test documentation build `, degraded here
llvm#83149 .
@owenca owenca removed the clang Clang issues not falling into any other category label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants