[all-commits] [llvm/llvm-project] 13316a: [clang] Disable a few formatting options for test/

Bruno Ricci via All-commits all-commits at lists.llvm.org
Sun Jul 19 09:24:13 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 13316a77053514be552a0dad932e3455413b4f82
      https://github.com/llvm/llvm-project/commit/13316a77053514be552a0dad932e3455413b4f82
  Author: Bruno Ricci <riccibrun at gmail.com>
  Date:   2020-07-19 (Sun, 19 Jul 2020)

  Changed paths:
    M clang/test/.clang-format

  Log Message:
  -----------
  [clang] Disable a few formatting options for test/

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.

Differential Revision: https://reviews.llvm.org/D83901


  Commit: 89ff9bf061b4985d11cd4785958d8f8156d10f5d
      https://github.com/llvm/llvm-project/commit/89ff9bf061b4985d11cd4785958d8f8156d10f5d
  Author: Bruno Ricci <riccibrun at gmail.com>
  Date:   2020-07-19 (Sun, 19 Jul 2020)

  Changed paths:
    M clang/lib/Sema/SemaStmt.cpp
    M clang/test/SemaCXX/constant-expression-cxx11.cpp
    M clang/test/SemaCXX/consteval-return-void.cpp

  Log Message:
  -----------
  [clang] Fix the warning for a non-void consteval function without a return value to actually say "consteval".

This warning was modified in 796ed03b8412 to use the term "consteval"
for consteval functions. However the warning has never worked as
intended since the diagnostic's arguments are used in the wrong order.

This was unfortunately missed by 796ed03b8412 since no test did exercise
this specific warning.

Additionally send the NamedDecl* into the diagnostic instead of just the
IdentifierInfo* to correctly work with special names and template
arguments.


Compare: https://github.com/llvm/llvm-project/compare/d41d952be977...89ff9bf061b4


More information about the All-commits mailing list