From 1fdf6c82eb853692d90f4904480be88ce5feec79 Mon Sep 17 00:00:00 2001 From: Stephen Kelly Date: Mon, 10 Dec 2018 20:53:32 +0000 Subject: [PATCH] Change InitListExpr dump to label and pointer Summary: Don't add a child just for the label. Reviewers: aaron.ballman Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D55495 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@348794 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/AST/ASTDumper.cpp | 9 +++++---- test/AST/ast-dump-stmt.cpp | 5 ++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/AST/ASTDumper.cpp b/lib/AST/ASTDumper.cpp index e071575a1e..1a79137adb 100644 --- a/lib/AST/ASTDumper.cpp +++ b/lib/AST/ASTDumper.cpp @@ -1951,11 +1951,12 @@ void ASTDumper::VisitInitListExpr(const InitListExpr *ILE) { OS << " field "; NodeDumper.dumpBareDeclRef(Field); } + if (auto *Filler = ILE->getArrayFiller()) { - dumpChild([=] { - OS << "array filler"; - dumpStmt(Filler); - }); + OS << " array_filler"; + NodeDumper.dumpPointer(Filler); + + dumpStmt(Filler); } } diff --git a/test/AST/ast-dump-stmt.cpp b/test/AST/ast-dump-stmt.cpp index 671bdd6454..8548da72d1 100644 --- a/test/AST/ast-dump-stmt.cpp +++ b/test/AST/ast-dump-stmt.cpp @@ -90,9 +90,8 @@ void TestUnionInitList() { U us[3] = {1}; // CHECK: VarDecl {{.+}} col:5 us 'U [3]' cinit -// CHECK-NEXT: `-InitListExpr {{.+}} 'U [3]' -// CHECK-NEXT: |-array filler -// CHECK-NEXT: | `-InitListExpr {{.+}} 'U' field Field {{.+}} 'i' 'int' +// CHECK-NEXT: `-InitListExpr {{.+}} 'U [3]' array_filler 0x{{.+}} +// CHECK-NEXT: |-InitListExpr {{.+}} 'U' field Field {{.+}} 'i' 'int' // CHECK-NEXT: `-InitListExpr {{.+}} 'U' field Field {{.+}} 'i' 'int' // CHECK-NEXT: `-IntegerLiteral {{.+}} 'int' 1 } -- 2.40.0