[PATCH] D64454: [clang-tidy] Adding static analyzer check to list of clang-tidy checks

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

In D64454#1598043 <https://reviews.llvm.org/D64454#1598043>, @Nathan-Huckleberry wrote:

> In D64454#1587102 <https://reviews.llvm.org/D64454#1587102>, @aaron.ballman wrote:
>
> > I think this looks reasonable to me, though I am still not certain if the relative path in the python script will work with both the svn in-tree directory layout as well as the git monorepo layout (which I'm far less familiar with).
>
>
> This works in the git monorepo layout as that's what I'm using. Not sure if it works with the svn in-tree directory layout.


I use svn in-tree, so I tried it out and it does not seem to work for me.

  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 120, in <module>
      main()
    File "gen-static-analyzer-docs.py", line 109, in main
      checkers = get_checkers()
    File "gen-static-analyzer-docs.py", line 23, 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


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64454/new/

https://reviews.llvm.org/D64454