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

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

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

[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



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

[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/

https://reviews.llvm.org/D73966



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

[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 happy to do it for you.


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

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

[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: clang/docs/ReleaseNotes.rst
===
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -402,10 +402,24 @@
 Static Analyzer
 ---
 
-- The Clang analyzer checker ``DeadStores`` gets a new option called
-  ``WarnForDeadNestedAssignments`` to detect nested dead assignments
-  (enabled by default).
-- ...
+- New checker: ``alpha.cplusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
+
+- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia
+  handles.
+
+- New checker: ``security.insecureAPI.decodeValueOfObjCType`` warns about
+  potential buffer overflows when using ``[NSCoder decodeValueOfObjCType:at:]``
+
+- ``deadcode.DeadStores`` now warns about nested dead stores.
+
+- Condition values that are relevant to the occurance of a bug are far better
+  explained in bug reports.
+
+- Despite still being at an alpha stage, checkers implementing taint analyses
+  and C++ iterator rules were improved greatly.
+
+- Numerous smaller fixes.
 
 .. _release-notes-ubsan:
 


Index: clang/docs/ReleaseNotes.rst
===
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -402,10 +402,24 @@
 Static Analyzer
 ---
 
-- The Clang analyzer checker ``DeadStores`` gets a new option called
-  ``WarnForDeadNestedAssignments`` to detect nested dead assignments
-  (enabled by default).
-- ...
+- New checker: ``alpha.cplusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
+
+- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia
+  handles.
+
+- New checker: ``security.insecureAPI.decodeValueOfObjCType`` warns about
+  potential buffer overflows when using ``[NSCoder decodeValueOfObjCType:at:]``
+
+- ``deadcode.DeadStores`` now warns about nested dead stores.
+
+- Condition values that are relevant to the occurance of a bug are far better
+  explained in bug reports.
+
+- Despite still being at an alpha stage, checkers implementing taint analyses
+  and C++ iterator rules were improved greatly.
+
+- Numerous smaller fixes.
 
 .. _release-notes-ubsan:
 
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

[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


Index: clang/docs/ReleaseNotes.rst
===
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -402,31 +402,24 @@
 Static Analyzer
 ---
 
+- New checker: ``alpha.cplusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
+
 - New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia
   handles.
 
-- New checker: ``alpha.plusplus.PlacementNew`` to detect whether the storage
-  provided for default placement new is sufficiently large.
+- New checker: ``security.insecureAPI.decodeValueOfObjCType`` warns about
+  potential buffer overflows when using ``[NSCoder decodeValueOfObjCType:at:]``
 
-- The Clang analyzer checker ``DeadStores`` gets a new option called
-  ``WarnForDeadNestedAssignments`` to detect nested dead assignments
-  (enabled by default).
+- ``deadcode.DeadStores`` now warns about nested dead stores.
 
-- Condition values that greatly affect the occurance of a bug are now far 
better
-  explained in bug reports (further reading on the related
-  `GSoC'19 summary page `_).
+- Condition values that are relevant to the occurance of a bug are far better
+  explained in bug reports.
 
-- Despite still in being in alpha stage, checkers implementing taint analyses
+- Despite still being at an alpha stage, checkers implementing taint analyses
   and C++ iterator rules were improved greatly.
 
-- Analyses on LLVM's own source code are far more precise due to the modeling 
of
-  several LLVM specific techniques, like its custom RTTI, informing the 
analyzer
-  of the return values of core functions, and much more (further reading on the
-  related `GSoC'19 summary page 
`_)
 .
-
-- ObjectiveC++ changes:
-
-- Numerous smaller false positive fixes.
+- Numerous smaller fixes.
 
 .. _release-notes-ubsan:
 


Index: clang/docs/ReleaseNotes.rst
===
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -402,31 +402,24 @@
 Static Analyzer
 ---
 
+- New checker: ``alpha.cplusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
+
 - New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia
   handles.
 
-- New checker: ``alpha.plusplus.PlacementNew`` to detect whether the storage
-  provided for default placement new is sufficiently large.
+- New checker: ``security.insecureAPI.decodeValueOfObjCType`` warns about
+  potential buffer overflows when using ``[NSCoder decodeValueOfObjCType:at:]``
 
-- The Clang analyzer checker ``DeadStores`` gets a new option called
-  ``WarnForDeadNestedAssignments`` to detect nested dead assignments
-  (enabled by default).
+- ``deadcode.DeadStores`` now warns about nested dead stores.
 
-- Condition values that greatly affect the occurance of a bug are now far better
-  explained in bug reports (further reading on the related
-  `GSoC'19 summary page `_).
+- Condition values that are relevant to the occurance of a bug are far better
+  explained in bug reports.
 
-- Despite still in being in alpha stage, checkers implementing taint analyses
+- Despite still being at an alpha stage, checkers implementing taint analyses
   and C++ iterator rules were improved greatly.
 
-- Analyses on LLVM's own source code are far more precise due to the modeling of
-  several LLVM specific techniques, like its custom RTTI, informing the analyzer
-  of the return values of core functions, and much more (further reading on the
-  related `GSoC'19 summary page `_) .
-
-- ObjectiveC++ changes:
-
-- Numerous smaller false positive fixes.
+- Numerous smaller fixes.
 
 .. _release-notes-ubsan:
 
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

[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
> > 
> > 1) The checker is now in alpha.
> > 2) This checker wasn't enabled in the release either, as far as i 
> > understand.
> It was! This is about the HandleChecker the other is about the LockChecker.
Yup, this seems to be correct in the release notes.


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

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

[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 the release either, as far as i understand.
It was! This is about the HandleChecker the other is about the LockChecker.


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

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

[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 as i understand.


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

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

[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/

https://reviews.llvm.org/D73966

Files:
  clang/docs/ReleaseNotes.rst


Index: clang/docs/ReleaseNotes.rst
===
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -402,10 +402,31 @@
 Static Analyzer
 ---
 
+- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia
+  handles.
+
+- New checker: ``alpha.plusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
+
 - The Clang analyzer checker ``DeadStores`` gets a new option called
   ``WarnForDeadNestedAssignments`` to detect nested dead assignments
   (enabled by default).
-- ...
+
+- Condition values that greatly affect the occurance of a bug are now far 
better
+  explained in bug reports (further reading on the related
+  `GSoC'19 summary page `_).
+
+- Despite still in being in alpha stage, checkers implementing taint analyses
+  and C++ iterator rules were improved greatly.
+
+- Analyses on LLVM's own source code are far more precise due to the modeling 
of
+  several LLVM specific techniques, like its custom RTTI, informing the 
analyzer
+  of the return values of core functions, and much more (further reading on the
+  related `GSoC'19 summary page 
`_)
 .
+
+- ObjectiveC++ changes:
+
+- Numerous smaller false positive fixes.
 
 .. _release-notes-ubsan:
 


Index: clang/docs/ReleaseNotes.rst
===
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -402,10 +402,31 @@
 Static Analyzer
 ---
 
+- New checker: ``fuchsia.HandleChecker`` to detect leaks related to Fuchsia
+  handles.
+
+- New checker: ``alpha.plusplus.PlacementNew`` to detect whether the storage
+  provided for default placement new is sufficiently large.
+
 - The Clang analyzer checker ``DeadStores`` gets a new option called
   ``WarnForDeadNestedAssignments`` to detect nested dead assignments
   (enabled by default).
-- ...
+
+- Condition values that greatly affect the occurance of a bug are now far better
+  explained in bug reports (further reading on the related
+  `GSoC'19 summary page `_).
+
+- Despite still in being in alpha stage, checkers implementing taint analyses
+  and C++ iterator rules were improved greatly.
+
+- Analyses on LLVM's own source code are far more precise due to the modeling of
+  several LLVM specific techniques, like its custom RTTI, informing the analyzer
+  of the return values of core functions, and much more (further reading on the
+  related `GSoC'19 summary page `_) .
+
+- ObjectiveC++ changes:
+
+- Numerous smaller false positive fixes.
 
 .. _release-notes-ubsan:
 
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

[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:
> > 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