aaron.ballman added a comment.
Thank you for the fix! This generally LGTM, but I did find some extra test
cases to add.
================
Comment at: clang/test/AST/ast-print-method-decl.cpp:1
+// RUN: %clang_cc1 -ast-print %s -o - | FileCheck %s
+
----------------
I think the file should be named `ast-print-delegating-constructor.cpp`, WDYT?
================
Comment at: clang/test/AST/ast-print-method-decl.cpp:15
+ // CHECK-NEXT: };
+};
----------------
I'd also like to see test cases along the lines of:
```
struct B {
template <typename Ty>
B(Ty);
B(int X) : B((float)X) {}
};
struct C {
C(auto);
C(int) : C("") {}
};
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154186/new/
https://reviews.llvm.org/D154186
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
