[PATCH] D49245: [ASTImporter] Import implicit methods of existing class.

2018-07-16 Thread Balázs Kéri via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC337162: [ASTImporter] Import implicit methods of existing class. (authored by balazske, committed by ). Changed prior to commit: https://reviews.llvm.org/D49245?vs=155625=155648#toc Repository: rC

[PATCH] D49245: [ASTImporter] Import implicit methods of existing class.

2018-07-16 Thread Balázs Kéri via Phabricator via cfe-commits
balazske updated this revision to Diff 155625. balazske added a comment. - Small style and comment changes. Repository: rC Clang https://reviews.llvm.org/D49245 Files: lib/AST/ASTImporter.cpp unittests/AST/ASTImporterTest.cpp Index: unittests/AST/ASTImporterTest.cpp

[PATCH] D49245: [ASTImporter] Import implicit methods of existing class.

2018-07-14 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. LGTM. Just some stylish nits. To resolve this issue, I used `Sema::DeclareImplicit...` methods. But I like this approach much more because it doesn't allows to forget different kinds of

[PATCH] D49245: [ASTImporter] Import implicit methods of existing class.

2018-07-12 Thread Balázs Kéri via Phabricator via cfe-commits
balazske created this revision. Herald added subscribers: cfe-commits, martong. Herald added a reviewer: a.sidorin. Herald added a reviewer: a.sidorin. When an already existing class is encountered during import, check if it has implicit methods that are missing in the existing one, and import