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

Reply via email to