[PATCH] D64454: [clang-tidy] Adding static analyzer check to list of clang-tidy checks
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 1 12:23:56 PDT 2019
aaron.ballman added a comment.
This seems reasonable to me except for the fact that the script doesn't run when I try it locally.
c:\llvm\tools\clang\tools\extra\docs\clang-tidy\checks>python gen-static-analyzer-docs.py
Traceback (most recent call last):
File "gen-static-analyzer-docs.py", line 148, in <module>
main()
File "gen-static-analyzer-docs.py", line 137, in main
checkers = get_checkers(file_path)
File "gen-static-analyzer-docs.py", line 22, in get_checkers
result = subprocess.run(["llvm-tblgen", "--dump-json", "-I",
AttributeError: 'module' object has no attribute 'run'
c:\llvm\tools\clang\tools\extra\docs\clang-tidy\checks>python --version
Python 2.7.16
I don't think that API is available in Python 2.7, which is our current min required version of Python. That's the last outstanding item for the review that I can see.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64454/new/
https://reviews.llvm.org/D64454
More information about the cfe-commits
mailing list