Page MenuHomePhabricator
Differential D105447 Diff 356821 clang/utils/analyzer/entrypoint.py

Changeset View

Standalone View

View Options

clang/utils/analyzer/entrypoint.py

import argparse import argparse
import os import os
import sys import sys
from subprocess import call, check_call, CalledProcessError from subprocess import call, check_call, CalledProcessError
from time import sleep from time import sleep
from typing import List, Tuple from typing import List, Tuple
def main(): def main():
settings, rest = parse_arguments() settings, rest = parse_arguments()
cmake_opts = ['-D' + cmd for cmd in settings.D]
vsavchenkoUnsubmitted
Done
ReplyInline Actions

I think we should still use argparse for stuff like this, and we don't want any of these flags to sneak into rest.
https://stackoverflow.com/a/31141568/11582326

vsavchenko: I think we should still use `argparse` for stuff like this, and we don't want any of these…
manasAuthorUnsubmitted
Done
ReplyInline Actions

Understood! I will rework on this. Thanks for the review.

manas: Understood! I will rework on this. Thanks for the review.
if settings.wait: if settings.wait:
wait() wait()
if settings.build_llvm or settings.build_llvm_only: if settings.build_llvm or settings.build_llvm_only:
build_llvm() build_llvm(cmake_opts)
if settings.build_llvm_only: if settings.build_llvm_only:
return return
sys.exit(test(rest)) sys.exit(test(rest))
def wait(): def wait():
# It is an easy on CPU way of keeping the docker container running # It is an easy on CPU way of keeping the docker container running
# while the user has a terminal session in that container. # while the user has a terminal session in that container.
while True: while True:
sleep(3600) sleep(3600)
def parse_arguments() -> Tuple[argparse.Namespace, List[str]]: def parse_arguments() -> Tuple[argparse.Namespace, List[str]]:
parser = argparse.ArgumentParser() parser = argparse.ArgumentParser()
parser.add_argument('--wait', action='store_true') parser.add_argument('--wait', action='store_true')
parser.add_argument('--build-llvm', action='store_true') parser.add_argument('--build-llvm', action='store_true')
parser.add_argument('--build-llvm-only', action='store_true') parser.add_argument('--build-llvm-only', action='store_true')
parser.add_argument('-D', action='append', default=[])
return parser.parse_known_args() return parser.parse_known_args()
def build_llvm(): def build_llvm(cmake_options):
os.chdir('/build') os.chdir('/build')
try: try:
if is_cmake_needed(): if is_cmake_needed():
cmake() cmake(cmake_options)
ninja() ninja()
except CalledProcessError: except CalledProcessError:
print("Build failed!") print("Build failed!")
sys.exit(1) sys.exit(1)
def is_cmake_needed(): def is_cmake_needed():
return "build.ninja" not in os.listdir() return "build.ninja" not in os.listdir()
CMAKE_COMMAND = "cmake -G Ninja -DCMAKE_BUILD_TYPE=Release " \ CMAKE_COMMAND = "cmake -G Ninja -DCMAKE_BUILD_TYPE=Release " \
"-DCMAKE_INSTALL_PREFIX=/analyzer -DLLVM_TARGETS_TO_BUILD=X86 " \ "-DCMAKE_INSTALL_PREFIX=/analyzer -DLLVM_TARGETS_TO_BUILD=X86 " \
"-DLLVM_ENABLE_PROJECTS=\"clang;openmp\" -DLLVM_BUILD_RUNTIME=OFF " \ "-DLLVM_ENABLE_PROJECTS=\"clang;openmp\" -DLLVM_BUILD_RUNTIME=OFF " \
"-DLLVM_ENABLE_TERMINFO=OFF -DCLANG_ENABLE_ARCMT=OFF " \ "-DLLVM_ENABLE_TERMINFO=OFF -DCLANG_ENABLE_ARCMT=OFF " \
"-DCLANG_ENABLE_STATIC_ANALYZER=ON" "-DCLANG_ENABLE_STATIC_ANALYZER=ON"
def cmake(): def cmake(cmake_options):
check_call(CMAKE_COMMAND + ' /llvm-project/llvm', shell=True) check_call(CMAKE_COMMAND + ' '.join(cmake_options) + ' /llvm-project/llvm',
shell=True)
def ninja(): def ninja():
check_call("ninja install", shell=True) check_call("ninja install", shell=True)
def test(args: List[str]) -> int: def test(args: List[str]) -> int:
os.chdir("/projects") os.chdir("/projects")
return call("/scripts/SATest.py " + " ".join(args), shell=True) return call("/scripts/SATest.py " + " ".join(args), shell=True)
if __name__ == '__main__': if __name__ == '__main__':
main() main()