a_sidorin added a comment.

Hi Gabor,
This patch LGTM mostly, but there is a comment inline.



================
Comment at: include/clang/AST/ASTStructuralEquivalence.h:81
         EqKind(EqKind), StrictTypeSpelling(StrictTypeSpelling),
-        ErrorOnTagTypeMismatch(ErrorOnTagTypeMismatch), Complain(Complain) {}
+        Complain(Complain), ErrorOnTagTypeMismatch(ErrorOnTagTypeMismatch) {}
 
----------------
I see the argument order change but I don't see any callers changed. Do we 
really need this order change?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58897/new/

https://reviews.llvm.org/D58897



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to