[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