This is an archive of the discontinued LLVM Phabricator instance.

Differential D50731

[ASTImporter] Add test for ExprWithCleanups
ClosedPublic

Authored by teemperor on Aug 14 2018, 1:01 PM.

Details

Reviewers
a.sidorin
a_sidorin
Commits
rG380e372b13a5: [ASTImporter] Add test for ExprWithCleanups
rL339830: [ASTImporter] Add test for ExprWithCleanups
rC339830: [ASTImporter] Add test for ExprWithCleanups

Diff Detail

Event Timeline

teemperor created this revision.Aug 14 2018, 1:01 PM
Herald added a reviewer: a.sidorin. · View Herald TranscriptAug 14 2018, 1:01 PM
Herald added a subscriber: martong. · View Herald Transcript
a_sidorin accepted this revision.Aug 15 2018, 3:45 PM
a_sidorin added a subscriber: a_sidorin.

Tests are always welcome. Thanks!

This revision is now accepted and ready to land.Aug 15 2018, 3:45 PM
Closed by commit rC339830: [ASTImporter] Add test for ExprWithCleanups (authored by teemperor). · Explain WhyAug 15 2018, 3:52 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
test/
Import/
expr-with-cleanups/
Inputs/
8 lines
8 lines

Diff 160674

test/Import/expr-with-cleanups/Inputs/S.cpp

  • This file was added.
struct RAII {
int i = 0;
RAII() { i++; }
~RAII() { i--; }
};
void f() {
RAII();
}

test/Import/expr-with-cleanups/test.cpp

  • This file was added.
// RUN: clang-import-test -dump-ast -import %S/Inputs/S.cpp -expression %s | FileCheck %s
// CHECK: ExprWithCleanups
// CHECK-SAME: 'RAII'
// CHECK-NEXT: CXXBindTemporaryExpr
void expr() {
f();
}