gamesh411 added inline comments.

================
Comment at: include/clang/AST/ASTStructuralEquivalence.h:81
         EqKind(EqKind), StrictTypeSpelling(StrictTypeSpelling),
-        ErrorOnTagTypeMismatch(ErrorOnTagTypeMismatch), Complain(Complain) {}
+        Complain(Complain), ErrorOnTagTypeMismatch(ErrorOnTagTypeMismatch) {}
 
----------------
a_sidorin wrote:
> I see the argument order change but I don't see any callers changed. Do we 
> really need this order change?
Strictly speaking, there is no need for this change, the constructor param 
order does not match the initialization, but that is just a nit. I have updated 
the diff. Cheers :)


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