[PATCH] D73966: [analyzer] Add 10.0.0 release notes.

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 5 02:38:56 PST 2020


Szelethus added inline comments.


================
Comment at: clang/docs/ReleaseNotes.rst:408
+
+- New checker: ``alpha.plusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
----------------
Charusso wrote:
> balazske wrote:
> > Szelethus wrote:
> > > NoQ wrote:
> > > > I think this one's out of alpha already (i.e., `cplusplus.PlacementNew`). I'd also bump it to the top because it applies to more users.
> > > Yup, but AFAIK it moved out of alpha after rc1 was tagged.
> > `alpha.cplusplus.PlacementNew` not `alpha.plusplus.PlacementNew`?
> `cplusplus.PlacementNew`
> (cf. https://github.com/llvm/llvm-project/commit/bc29069dc40)
Nice catch ;) I checked out the `llvmorg-10.0.0-rc1` tag and can confirm that at that point, the checker is still in alpha, unfortunately.


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

https://reviews.llvm.org/D73966





More information about the cfe-commits mailing list