a.sidorin accepted this revision. a.sidorin added a comment. This revision is now accepted and ready to land.
Hi Gabor, LGTM, thank you for addressing the comments! Just a minor nit, it's OK to fix it before committing without a separate review. ================ Comment at: unittests/AST/StructuralEquivalenceTest.cpp:67 + bool testStructuralMatch(std::tuple<NamedDecl *, NamedDecl *> t) { + using std::get; + return testStructuralMatch(get<0>(t), get<1>(t)); ---------------- Not sure we need this using: we can move up the `using` below or just write std::get twice. Repository: rC Clang https://reviews.llvm.org/D46867 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits