[all-commits] [llvm/llvm-project] a3da62: [clang][DeclPrinter] Fix missing semicolon in AST ...

[Timo Stripf via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a3da6284c23affdd9092b2641017e99d85c2d89b
      https://github.com/llvm/llvm-project/commit/a3da6284c23affdd9092b2641017e99d85c2d89b
  Author: Timo Stripf <timo.stripf at emmtrix.com>
  Date:   2023-07-28 (Fri, 28 Jul 2023)

  Changed paths:
    M clang/lib/AST/DeclPrinter.cpp
    M clang/test/AST/ast-print-method-decl.cpp

  Log Message:
  -----------
  [clang][DeclPrinter] Fix missing semicolon in AST print for methods that are definitions without having a body

DeclPrinter used FunctionDecl::isThisDeclarationADefinition to decide if the decl requires a semicolon at the end. However, there are several methods without body (that require a semicolon) that are definitions.

Fixes https://github.com/llvm/llvm-project/issues/62996

Reviewed By: aaron.ballman

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