NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

I approve the patch and i don't see anything obvious that we're missing out 
(@Szelethus, your GSoC isn't on by default back in 9.0, only in master, right?).

I guess we should really try harder to fill it in with every patch that 
deserves it.



================
Comment at: clang/docs/ReleaseNotes.rst:235
+
+- New checker: 'optin.osx.OSObjectCStyleCast' to find C-style casts of 
OSObjects
+
----------------
"of XNU libkern OSObjects", so that it sounded less esoteric :/ 


================
Comment at: clang/docs/ReleaseNotes.rst:237-238
+
+- New package: 'apiModeling.llvm' contains modeling checkers to improve the
+  accuracy of reports on LLVM's codebase.
+
----------------
NoQ wrote:
> "On LLVM's **own** codebase" :)
> 
> @Charusso: I think we need to cherry-pick all of our latest patches to this 
> checker to 9.0 because otherwise it'll be crashing with the 
> `isValidBaseClass()` assertion.
>>! In D66765#1645726, @Szelethus wrote:
> Please note that LLVM 9.0.0-final is due on the 28th of August.

Mmm, most likely too late.


================
Comment at: clang/docs/ReleaseNotes.rst:251
+
+  - ``-analyzer-checker-help-alpha``: The list of incomplet and inkorrekt
+    checkers not yet advised to be turned on.
----------------
xbolva00 wrote:
> Please fix this
I think this was supposed to be a joke but i don't insist^^


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66765



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

Reply via email to