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

2020-02-10 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus closed this revision. Szelethus added a comment. Cheers! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73966/new/ https://reviews.llvm.org/D73966 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

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

2020-02-10 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. Pushed to 10.x in 64515b35844b925bdb76821d03ad4d7ddebe06e7 . Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73966/new/ https://reviews.llvm.org/D73966

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

2020-02-10 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I missed out on the transition to github, so I suspect that the commit access will only be extended to it after tagging rc2. I think it would be better if you committed this on my behalf, thanks! :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73966/new/

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

2020-02-10 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. Herald added a subscriber: steakhal. Looks good to me. Thanks for writing release notes! If you have commit access, go ahead and push directly to the branch, otherwise let me know and I'll be

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

2020-02-06 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus updated this revision to Diff 242869. Szelethus added a comment. //Actually//, **actually** upload the correct one. Getting rusty eh. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73966/new/ https://reviews.llvm.org/D73966 Files: clang/docs/ReleaseNotes.rst Index:

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

2020-02-06 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus updated this revision to Diff 242860. Szelethus marked an inline comment as done. Szelethus added a comment. //Actually// update the revision. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73966/new/ https://reviews.llvm.org/D73966 Files: clang/docs/ReleaseNotes.rst

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

2020-02-06 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus marked 3 inline comments as done. Szelethus added inline comments. Comment at: clang/docs/ReleaseNotes.rst:405 +- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia + handles. xazax.hun wrote: > NoQ wrote: > > D74004 > > > >

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

2020-02-05 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added inline comments. Comment at: clang/docs/ReleaseNotes.rst:405 +- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia + handles. NoQ wrote: > D74004 > > 1) The checker is now in alpha. > 2) This checker wasn't enabled in

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

2020-02-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added inline comments. Comment at: clang/docs/ReleaseNotes.rst:405 +- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia + handles. D74004 1) The checker is now in alpha. 2) This checker wasn't enabled in the release either, as far

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

2020-02-05 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus updated this revision to Diff 242539. Szelethus marked 10 inline comments as done. Szelethus retitled this revision from "[analyzer][WIP] Add 10.0.0 release notes." to "[analyzer] Add 10.0.0 release notes.". CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73966/new/

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

2020-02-05 Thread Kristóf Umann via Phabricator via cfe-commits
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: > >