[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