jyknight added inline comments.
================ Comment at: clang/lib/AST/ItaniumMangle.cpp:5145 + ASTContext &Ctx = Context.getASTContext(); + if (Ctx.getLangOpts().getClangABICompat() > LangOptions::ClangABI::Ver11) { + mangleExpression(E, UnknownArity, /*AsTemplateArg=*/true); ---------------- rsmith wrote: > Given that we've now branched for the Clang 12 release, please make sure you > either backport this to the Clang 12 release branch or bump this to `> Ver12`. I think this series of patches should be pushed to clang 12, and will thus keep the version as-is, and ensure it makes it into the newly-created branch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95487/new/ https://reviews.llvm.org/D95487 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits