giannicrivello added a comment. In D138964#3958986 <https://reviews.llvm.org/D138964#3958986>, @NoQ wrote:
> Nice, thanks! > > For non-trivial patches it's recommended to include full diff context via > `-U999999` > (https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface) > so that reviewers could easily see what does the surrounding code do. Ok will do, thanks. So how would I move forward from here? ================ Comment at: clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td:30 -/// Alpha entries are under development, might be incomplet, inkorrekt and /// unstable. ---------------- NoQ wrote: > This one's probably intentional! I see what they did there. lol Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138964/new/ https://reviews.llvm.org/D138964 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits