[PATCH] D139604: [PATCH] Github Issue: Create a check that warns about using %p printf specifier #43453

2022-12-07 Thread Gianni Crivello via Phabricator via cfe-commits
giannicrivello updated this revision to Diff 481154. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139604/new/ https://reviews.llvm.org/D139604 Files: clang/include/clang/StaticAnalyzer/Checkers/Checkers.td clang/lib/StaticAnalyzer/Checkers/UnixAPIChecker.cpp Index: clang/lib/Stati

[PATCH] D139604: [PATCH] Github Issue: Create a check that warns about using %p printf specifier #43453

2022-12-07 Thread Gianni Crivello via Phabricator via cfe-commits
giannicrivello created this revision. giannicrivello added reviewers: NoQ, tbaeder. giannicrivello added a project: clang. Herald added subscribers: steakhal, martong. Herald added a reviewer: sscalpone. Herald added projects: Flang, All. giannicrivello requested review of this revision. Herald add

[PATCH] D138964: Fixed a few spelling errors in clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

2022-11-30 Thread Gianni Crivello via Phabricator via cfe-commits
giannicrivello updated this revision to Diff 478960. giannicrivello added a comment. Addressed the intentional spelling errors, also fixed a few more grammar errors Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138964/new/ https://reviews.llvm.org/

[PATCH] D139017: Changed the intentional spelling errors back to the original, made more slight grammar corrections

2022-11-30 Thread Gianni Crivello via Phabricator via cfe-commits
giannicrivello created this revision. Herald added a subscriber: steakhal. Herald added a reviewer: NoQ. Herald added a project: All. giannicrivello requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo http

[PATCH] D138964: Fixed a few spelling errors in clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

2022-11-29 Thread Gianni Crivello via Phabricator via cfe-commits
giannicrivello added a comment. In D138964#3958986 , @NoQ wrote: > Nice, thanks! > > For non-trivial patches it's recommended to include full diff context via > `-U99` > (https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interfa

[PATCH] D138963: Fixed a few spelling errors.

2022-11-29 Thread Gianni Crivello via Phabricator via cfe-commits
giannicrivello created this revision. Herald added a subscriber: steakhal. Herald added a reviewer: NoQ. Herald added a project: All. giannicrivello requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo http