aaron.ballman added a comment. In D128745#3851225 <https://reviews.llvm.org/D128745#3851225>, @ychen wrote:
> As a next step, I'll remove the ClangABICompat checks for these DRs (make > these DRs effective unconditionally). If we saw proof that these have > deployment difficulties. We can (1) re-run the rules with the committee as > suggested by @rjmccall; (2) consider alternatives (including reverting these > DRs) based on the feedback. Please let me know if you object to this or have > any other concerns. This sounds like a good plan to me. When you remove the ABI compat checks, please be sure to add the clang-vendors group to the review and a release note to the potentially breaking changes section. Once that lands, you should also make an announcement in Discourse. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128745/new/ https://reviews.llvm.org/D128745 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
