[PATCH] D155445: [analyzer][docs] Add CSA release notes

[Balázs Benics via Phabricator via cfe-commits]

steakhal added inline comments.


================
Comment at: clang/docs/ReleaseNotes.rst:907
+  Any use of this flag will result in an error.
+  (`7cd1f3ad22e4 <https://github.com/llvm/llvm-project/commit/7cd1f3ad22e4>`_)
+- Fixed a null-pointer dereference crash inside the ``MoveChecker``.
----------------
xazax.hun wrote:
> I think we should mention  something like "Use -fstrict-flex-array=<N> instead if necessary."
Good point.


================
Comment at: clang/docs/ReleaseNotes.rst:908
+  (`7cd1f3ad22e4 <https://github.com/llvm/llvm-project/commit/7cd1f3ad22e4>`_)
+- Fixed a null-pointer dereference crash inside the ``MoveChecker``.
+  (`d172b65ef001 <https://github.com/llvm/llvm-project/commit/d172b65ef001>`_)
----------------
xazax.hun wrote:
> I think we usually do not mention crash fixes in the changelog. We have them in almost every release and sometimes there are quite a few of them.
I won't mention the explicit commit where it was fixed.
However, downstream users might wanna know about crashes and fixes that happened in this release.
And speaking about past practices about release notes, I think we can improve on that TBH.
We can move it down on the list if you want, but I'd rather keep it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155445/new/

https://reviews.llvm.org/D155445