[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-05-03 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. In D60925#1489034 , @o.gyorgy wrote: > Great, at least the users will not enable the debug checkers by accident! > We will check on the CodeChecker side if any configuration needs to be > updated. > > LGTM I committed after c

[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-05-02 Thread Gyorgy Orban via Phabricator via cfe-commits
o.gyorgy added a comment. Great, at least the users will not enable the debug checkers by accident! We will check on the CodeChecker side if any configuration needs to be updated. LGTM Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60925/new/ https://reviews.llvm.

[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-05-01 Thread Kristóf Umann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL359720: [analyzer] Don't display implementation checkers under -analyzer-checker-help… (authored by Szelethus, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Change

[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-05-01 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus updated this revision to Diff 197609. Szelethus edited the summary of this revision. Szelethus added a comment. Hide `security.insecureAPI.SecuritySyntaxChecker` by default. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60925/new/ https://reviews.llvm.org/D60925 Files: incl

[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-04-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I plan to do the same for non-checker and checker options too. I'll leave this up for just a little while to give people time to object. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60925/new/ https://reviews.llvm.org/D60925 __

[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-04-22 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. Yup, that sounds useful! > I intentionally chose not to hide alpha checkers, because they have a scary > enough name, in my opinion, to cause no surprise when they emit false > positives or cause crashes. I've seen people yelling really loudly, like "look, your clang is cr

[PATCH] D60925: [analyzer] Don't display implementation checkers under -analyzer-checker-help, but do under the new flag -analyzer-checker-help-hidden

2019-04-19 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus created this revision. Szelethus added reviewers: NoQ, dcoughlin, baloghadamsoftware, Charusso, xazax.hun, rnkovacs. Szelethus added a project: clang. Herald added subscribers: cfe-commits, gamesh411, dkrupp, donat.nagy, jfb, mikhail.ramalho, a.sidorin, JDevlieghere, szepet, whisperity.