[PATCH] D45451: [ItaniumMangle] Undeduced auto type doesn't belong in the substitution table

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 21 05:56:52 PDT 2018


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





More information about the cfe-commits mailing list