[PATCH] D136120: [Clang] follow-up D128745, remove ClangABICompat checks

This revision was automatically updated to reflect the committed changes. Closed by commit rG40e99473170f: [Clang] follow-up D128745, remove ClangABICompat checks (authored by ychen). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136120/new/ https:/

[PATCH] D136120: [Clang] follow-up D128745, remove ClangABICompat checks

ychen updated this revision to Diff 472727. ychen added a comment. - rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136120/new/ https://reviews.llvm.org/D136120 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaTemplateDeduction.cpp

[PATCH] D136120: [Clang] follow-up D128745, remove ClangABICompat checks

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136120/new/ https://reviews.llvm.org/D136120 _

[PATCH] D136120: [Clang] follow-up D128745, remove ClangABICompat checks

ychen added a comment. ping? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136120/new/ https://reviews.llvm.org/D136120 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[PATCH] D136120: [Clang] follow-up D128745, remove ClangABICompat checks

ychen updated this revision to Diff 468630. ychen added a comment. - update Release notes - remove obsolete CodeGen tests which was used to test ClangABICompat checks, similar Sema tests are already in place. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D136120: [Clang] follow-up D128745, remove ClangABICompat checks

tahonermann added a comment. The code changes look good to me. I offered a suggested rewording of the release note. Comment at: clang/docs/ReleaseNotes.rst:499-501 +- Implemented DR692, DR1395 and DR1432. Note that the fix for DR1432 is speculative + that there is no wording