aaron.ballman added a comment. Continues to LGTM, though I had a recommendation for the release note. Feel free to land when you'd like.
================ Comment at: clang/docs/ReleaseNotes.rst:54 +------------------ +- ``CXXNewExpr::getArraySize()`` previously returned a ``llvm::Optional`` + wrapping a ``nullptr`` when the ``CXXNewExpr`` did not have an array ---------------- Just added a reference to the bug that was fixed. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119525/new/ https://reviews.llvm.org/D119525 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
