aaron.ballman accepted this revision. aaron.ballman added a subscriber: tstellar. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM! ================ Comment at: clang/docs/ReleaseNotes.rst:74 number of arguments cause an assertion fault. +- Fix `#57151 <https://github.com/llvm/llvm-project/issues/57151>`_. + ``-Wcomma`` is emitted for void returning functions. ---------------- inclyc wrote: > Do we need an NFC commit to unify the format of issues mentioned here? (Maybe > it can be scheduled for a later release). The different ways of citing issues > here can be confusing. `Issue xxxx` `# xxx` in this context. If we wanted to do something like that, I think we should probably do that as part of the release process as a checklist item (CC @tstellar) but I don't have strong opinions either. We can certainly do it with NFC fixes as folks notice issues if we're fine still having some inconsistency when we release (which I'm fine with, personally). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131892/new/ https://reviews.llvm.org/D131892 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
