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