hokein added a comment. Thanks for the review.
In D150955#4356580 <https://reviews.llvm.org/D150955#4356580>, @aaron.ballman wrote: > LGTM assuming CI is okay with it -- the patch did not apply so precommit CI > never ran. I'm surprised no tests break as a result of this because it > changes both C and C++ behavior. The changes should have a release note, too. This patch was based on my another fix (this probably explains why it failed to apply on pre-merge testing CI). I have test it with `ninja check-clang`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150955/new/ https://reviews.llvm.org/D150955 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
