a.sidorin accepted this revision.
a.sidorin added a comment.

Refactoring changes are always appreciated. I only have a minor naming nit.



================
Comment at: include/clang/AST/ASTStructuralEquivalence.h:33
+  /// AST contexts for which we are checking structural equivalence.
+  ASTContext &C1, &C2;
+
----------------
FromCtx, ToCtx?


https://reviews.llvm.org/D31777



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D31777: [ASTImport... Sean Callanan via Phabricator via cfe-commits
    • [PATCH] D31777: [ASTI... Aleksei Sidorin via Phabricator via cfe-commits

Reply via email to