| Message ID | 20230923053515.535607-4-irogers@google.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | BPF |
| Headers | show |
| Series | clang-tools support in tools | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On Fri, Sep 22, 2023 at 10:35 PM Ian Rogers <irogers@google.com> wrote: > > Add a -checks argument to allow the checks passed to the clang-tool to > be set on the command line. > > Add a pass through -header-filter option. > > Signed-off-by: Ian Rogers <irogers@google.com> > --- > scripts/clang-tools/run-clang-tools.py | 34 ++++++++++++++++++++------ > 1 file changed, 27 insertions(+), 7 deletions(-) > > diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py > index 3266708a8658..5dfe03852cb4 100755 > --- a/scripts/clang-tools/run-clang-tools.py > +++ b/scripts/clang-tools/run-clang-tools.py > @@ -33,6 +33,11 @@ def parse_arguments(): > path_help = "Path to the compilation database to parse" > parser.add_argument("path", type=str, help=path_help) > > + checks_help = "Checks to pass to the analysis" > + parser.add_argument("-checks", type=str, default=None, help=checks_help) > + header_filter_help = "Pass the -header-filter value to the tool" > + parser.add_argument("-header-filter", type=str, default=None, help=header_filter_help) > + > return parser.parse_args() > > > @@ -45,14 +50,29 @@ def init(l, a): > > def run_analysis(entry): > # Disable all checks, then re-enable the ones we want > - checks = [] > - checks.append("-checks=-*") > - if args.type == "clang-tidy": > - checks.append("linuxkernel-*") > + global args > + checks = None > + if args.checks: > + checks = args.checks.split(',') > else: > - checks.append("clang-analyzer-*") > - checks.append("-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling") > - p = subprocess.run(["clang-tidy", "-p", args.path, ",".join(checks), entry["file"]], > + checks = ["-*"] > + if args.type == "clang-tidy": > + checks.append("linuxkernel-*") > + else: > + checks.append("clang-analyzer-*") > + checks.append("-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling") > + file = entry["file"] > + if not file.endswith(".c") and not file.endswith(".cpp"): > + with lock: > + print(f"Skipping non-C file: '{file}'", file=sys.stderr) > + return > + pargs = ["clang-tidy", "-p", args.path] > + if checks: ^ can `checks` ever be falsy here? I don't think we need this conditional check? > + pargs.append("-checks=" + ",".join(checks)) > + if args.header_filter: > + pargs.append("-header-filter=" + args.header_filter) > + pargs.append(file) > + p = subprocess.run(pargs, > stdout=subprocess.PIPE, > stderr=subprocess.STDOUT, > cwd=entry["directory"]) > -- > 2.42.0.515.g380fc7ccd1-goog >
On Mon, Sep 25, 2023 at 8:44 AM Nick Desaulniers <ndesaulniers@google.com> wrote: > > On Fri, Sep 22, 2023 at 10:35 PM Ian Rogers <irogers@google.com> wrote: > > > > Add a -checks argument to allow the checks passed to the clang-tool to > > be set on the command line. > > > > Add a pass through -header-filter option. > > > > Signed-off-by: Ian Rogers <irogers@google.com> > > --- > > scripts/clang-tools/run-clang-tools.py | 34 ++++++++++++++++++++------ > > 1 file changed, 27 insertions(+), 7 deletions(-) > > > > diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py > > index 3266708a8658..5dfe03852cb4 100755 > > --- a/scripts/clang-tools/run-clang-tools.py > > +++ b/scripts/clang-tools/run-clang-tools.py > > @@ -33,6 +33,11 @@ def parse_arguments(): > > path_help = "Path to the compilation database to parse" > > parser.add_argument("path", type=str, help=path_help) > > > > + checks_help = "Checks to pass to the analysis" > > + parser.add_argument("-checks", type=str, default=None, help=checks_help) > > + header_filter_help = "Pass the -header-filter value to the tool" > > + parser.add_argument("-header-filter", type=str, default=None, help=header_filter_help) > > + > > return parser.parse_args() > > > > > > @@ -45,14 +50,29 @@ def init(l, a): > > > > def run_analysis(entry): > > # Disable all checks, then re-enable the ones we want > > - checks = [] > > - checks.append("-checks=-*") > > - if args.type == "clang-tidy": > > - checks.append("linuxkernel-*") > > + global args > > + checks = None > > + if args.checks: > > + checks = args.checks.split(',') > > else: > > - checks.append("clang-analyzer-*") > > - checks.append("-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling") > > - p = subprocess.run(["clang-tidy", "-p", args.path, ",".join(checks), entry["file"]], > > + checks = ["-*"] > > + if args.type == "clang-tidy": > > + checks.append("linuxkernel-*") > > + else: > > + checks.append("clang-analyzer-*") > > + checks.append("-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling") > > + file = entry["file"] > > + if not file.endswith(".c") and not file.endswith(".cpp"): > > + with lock: > > + print(f"Skipping non-C file: '{file}'", file=sys.stderr) > > + return > > + pargs = ["clang-tidy", "-p", args.path] > > + if checks: > > ^ can `checks` ever be falsy here? I don't think we need this > conditional check? Agreed, it was for an option that I removed. Will fix in v2. Thanks, Ian > > + pargs.append("-checks=" + ",".join(checks)) > > + if args.header_filter: > > + pargs.append("-header-filter=" + args.header_filter) > > + pargs.append(file) > > + p = subprocess.run(pargs, > > stdout=subprocess.PIPE, > > stderr=subprocess.STDOUT, > > cwd=entry["directory"]) > > -- > > 2.42.0.515.g380fc7ccd1-goog > > > > > -- > Thanks, > ~Nick Desaulniers
diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py index 3266708a8658..5dfe03852cb4 100755 --- a/scripts/clang-tools/run-clang-tools.py +++ b/scripts/clang-tools/run-clang-tools.py @@ -33,6 +33,11 @@ def parse_arguments(): path_help = "Path to the compilation database to parse" parser.add_argument("path", type=str, help=path_help) + checks_help = "Checks to pass to the analysis" + parser.add_argument("-checks", type=str, default=None, help=checks_help) + header_filter_help = "Pass the -header-filter value to the tool" + parser.add_argument("-header-filter", type=str, default=None, help=header_filter_help) + return parser.parse_args() @@ -45,14 +50,29 @@ def init(l, a): def run_analysis(entry): # Disable all checks, then re-enable the ones we want - checks = [] - checks.append("-checks=-*") - if args.type == "clang-tidy": - checks.append("linuxkernel-*") + global args + checks = None + if args.checks: + checks = args.checks.split(',') else: - checks.append("clang-analyzer-*") - checks.append("-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling") - p = subprocess.run(["clang-tidy", "-p", args.path, ",".join(checks), entry["file"]], + checks = ["-*"] + if args.type == "clang-tidy": + checks.append("linuxkernel-*") + else: + checks.append("clang-analyzer-*") + checks.append("-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling") + file = entry["file"] + if not file.endswith(".c") and not file.endswith(".cpp"): + with lock: + print(f"Skipping non-C file: '{file}'", file=sys.stderr) + return + pargs = ["clang-tidy", "-p", args.path] + if checks: + pargs.append("-checks=" + ",".join(checks)) + if args.header_filter: + pargs.append("-header-filter=" + args.header_filter) + pargs.append(file) + p = subprocess.run(pargs, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, cwd=entry["directory"])
Add a -checks argument to allow the checks passed to the clang-tool to be set on the command line. Add a pass through -header-filter option. Signed-off-by: Ian Rogers <irogers@google.com> --- scripts/clang-tools/run-clang-tools.py | 34 ++++++++++++++++++++------ 1 file changed, 27 insertions(+), 7 deletions(-)