This is an archive of the discontinued LLVM Phabricator instance.

Differential D50810

[ASTImporter] Add test for DoStmt
ClosedPublic

Authored by teemperor on Aug 15 2018, 2:31 PM.

Details

Reviewers
a.sidorin
martong
Commits
rG5fd7f152c8fc: [ASTImporter] Add test for DoStmt
rC339917: [ASTImporter] Add test for DoStmt
rL339917: [ASTImporter] Add test for DoStmt

Diff Detail

Repository
rL LLVM

Event Timeline

teemperor created this revision.Aug 15 2018, 2:31 PM
Herald added a reviewer: a.sidorin. · View Herald TranscriptAug 15 2018, 2:31 PM
Herald added a subscriber: martong. · View Herald Transcript
martong accepted this revision.Aug 16 2018, 12:28 AM

LGTM.

This revision is now accepted and ready to land.Aug 16 2018, 12:28 AM
Herald added a subscriber: rnkovacs. · View Herald TranscriptAug 16 2018, 12:28 AM
Closed by commit rL339917: [ASTImporter] Add test for DoStmt (authored by teemperor). · Explain WhyAug 16 2018, 11:21 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: llvm-commits. · View Herald TranscriptAug 16 2018, 11:21 AM

Revision Contents

PathSize
cfe/
trunk/
test/
Import/
do-stmt/
Inputs/
7 lines
15 lines

Diff 161075

cfe/trunk/test/Import/do-stmt/Inputs/F.cpp

void f() {
do
;
while (true);
do {
} while (false);
}

cfe/trunk/test/Import/do-stmt/test.cpp

// RUN: clang-import-test -dump-ast -import %S/Inputs/F.cpp -expression %s | FileCheck %s
// CHECK: DoStmt
// CHECK-NEXT: NullStmt
// CHECK-NEXT: CXXBoolLiteralExpr
// CHECK-SAME: true
// CHECK: DoStmt
// CHECK-NEXT: CompoundStmt
// CHECK-NEXT: CXXBoolLiteralExpr
// CHECK-SAME: false
void expr() {
f();
}