a_sidorin accepted this revision. a_sidorin added a comment. This revision is now accepted and ready to land.
LGTM! ================ Comment at: unittests/AST/ASTImporterTest.cpp:4869 + + // FromPlus have a different TU, thus its DeclarationName is different too. + Res = LT.lookup(ToTU, FromPlus->getDeclName()); ---------------- I think this comment should also be put before ASSERT_NE. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57905/new/ https://reviews.llvm.org/D57905 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
