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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits