[PATCH] D50732: [ASTImporter] Add test for CXXDefaultInitExpr

2018-08-15 Thread Raphael Isemann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL339839: [ASTImporter] Add test for CXXDefaultInitExpr (authored by teemperor, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D50732?vs=160678&

[PATCH] D50732: [ASTImporter] Add test for CXXDefaultInitExpr

2018-08-15 Thread Aleksei Sidorin via Phabricator via cfe-commits
a_sidorin accepted this revision. a_sidorin added a comment. This revision is now accepted and ready to land. Thanks! > As a side note: It seems this test case actually reveals that we don't import > the body of Foo's destructor? This is strange. If you manage to find the reason, please notify

[PATCH] D50732: [ASTImporter] Add test for CXXDefaultInitExpr

2018-08-15 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor added a comment. As a side note: It seems this test case actually reveals that we don't import the body of Foo's destructor? test/Import/cxx-default-init-expr/Inputs/S.cpp:1:8: inline function 'Foo::~Foo' is not defined struct Foo { ^ test/Import/cxx-default-init-expr/I

[PATCH] D50732: [ASTImporter] Add test for CXXDefaultInitExpr

2018-08-14 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor created this revision. Herald added a subscriber: martong. Herald added a reviewer: a.sidorin. Repository: rC Clang https://reviews.llvm.org/D50732 Files: test/Import/cxx-default-init-expr/Inputs/S.cpp test/Import/cxx-default-init-expr/test.cpp Index: test/Import/cxx-default-in