This is an archive of the discontinued LLVM Phabricator instance.

[ASTImporter] Add test for CXXDefaultInitExpr
ClosedPublic

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

Diff Detail

Event Timeline

teemperor created this revision.Aug 14 2018, 1:23 PM

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/Inputs/S.cpp:5:8: used here
struct S {
       ^

This doesn't make the test itself fail, so I'll fix this in an upcoming patch.

a_sidorin accepted this revision.Aug 15 2018, 5:08 PM
a_sidorin added a subscriber: a_sidorin.

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 us!

This revision is now accepted and ready to land.Aug 15 2018, 5:08 PM
This revision was automatically updated to reflect the committed changes.