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
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits