This revision was automatically updated to reflect the committed changes.
Closed by commit rC340277: [ASTImporter] Adding some friend function related
unittests. (authored by balazske, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D49798?vs=160972&id=161715#toc
Repository:
balazske updated this revision to Diff 160972.
balazske added a comment.
- Renamed variables, corrected a variable type.
Repository:
rC Clang
https://reviews.llvm.org/D49798
Files:
unittests/AST/ASTImporterTest.cpp
Index: unittests/AST/ASTImporterTest.cpp
=
a_sidorin added a comment.
Hi Balasz,
Thank you for the fixes.
Comment at: unittests/AST/ASTImporterTest.cpp:2258
+ASSERT_EQ(lookup_res.size(), 0u);
+lookup_res = cast(FromTU)->noload_lookup(FromName);
+ASSERT_EQ(lookup_res.size(), 1u);
a_sidorin w
balazske updated this revision to Diff 160548.
balazske added a comment.
- Corrected the code in tests.
Repository:
rC Clang
https://reviews.llvm.org/D49798
Files:
unittests/AST/ASTImporterTest.cpp
Index: unittests/AST/ASTImporterTest.cpp
==
a_sidorin added a comment.
Hi Balazs,
The patch looks mostly fine.
Comment at: unittests/AST/ASTImporterTest.cpp:2256
+FirstDeclMatcher().match(FromTU, cxxRecordDecl());
+auto lookup_res = Class->noload_lookup(FromName);
+ASSERT_EQ(lookup_res.size(), 0u);
-
balazske created this revision.
Herald added subscribers: cfe-commits, martong.
Herald added a reviewer: a.sidorin.
Repository:
rC Clang
https://reviews.llvm.org/D49798
Files:
unittests/AST/ASTImporterTest.cpp
Index: unittests/AST/ASTImporterTest.cpp