[PATCH] D83901: [clang] Disable a few formatting options for test/

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 15 13:09:33 PDT 2020


riccibruno created this revision.
riccibruno added reviewers: aaron.ballman, jdoerfert, lebedev.ri, MyDeveloperDay.
riccibruno added a project: clang.
Herald added a subscriber: cfe-commits.

Hopefully this will make the bot a little less noisy. Rationale for each:

`AlignTrailingComments`: We don't want to force-align the various `expected-error` and friends.
`CommentPragmas`: Tell clang-format to leave the `// CHECK:` and the `// expected-` alone.
`AlwaysBreakTemplateDeclarations`: Templates in tests often have no break between the template-head and the declaration.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83901

Files:
  clang/test/.clang-format


Index: clang/test/.clang-format
===================================================================
--- clang/test/.clang-format
+++ clang/test/.clang-format
@@ -1,2 +1,5 @@
 BasedOnStyle: LLVM
 ColumnLimit: 0
+AlignTrailingComments: false
+CommentPragmas: "(^ ?CHECK|^ ?expected-)"
+AlwaysBreakTemplateDeclarations: No


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83901.278289.patch
Type: text/x-patch
Size: 320 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200715/36606de0/attachment.bin>


More information about the cfe-commits mailing list