rjmccall added inline comments.
================ Comment at: clang/lib/AST/ItaniumMangle.cpp:2342 + if (isa<AutoType>(Ty)) + return false; return true; ---------------- rjmccall wrote: > I agree with your analysis that this shouldn't be a substitution candidate. > However, I think this probably needs an ABI-compatibility guard. You should probably add a comment like "Prior to LLVM v6, Clang accidentally treated deduced auto types as substitution candidates." https://reviews.llvm.org/D45451 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
