[PATCH] D105447: [analyzer] Allow cmake options to be passed to satest container
Manas Gupta via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 6 14:32:55 PDT 2021
manas updated this revision to Diff 356821.
manas added a comment.
Pass commands through settings instead of rest
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105447/new/
https://reviews.llvm.org/D105447
Files:
clang/utils/analyzer/entrypoint.py
Index: clang/utils/analyzer/entrypoint.py
===================================================================
--- clang/utils/analyzer/entrypoint.py
+++ clang/utils/analyzer/entrypoint.py
@@ -9,10 +9,11 @@
def main():
settings, rest = parse_arguments()
+ cmake_opts = ['-D' + cmd for cmd in settings.D]
if settings.wait:
wait()
if settings.build_llvm or settings.build_llvm_only:
- build_llvm()
+ build_llvm(cmake_opts)
if settings.build_llvm_only:
return
sys.exit(test(rest))
@@ -30,14 +31,15 @@
parser.add_argument('--wait', action='store_true')
parser.add_argument('--build-llvm', action='store_true')
parser.add_argument('--build-llvm-only', action='store_true')
+ parser.add_argument('-D', action='append', default=[])
return parser.parse_known_args()
-def build_llvm():
+def build_llvm(cmake_options):
os.chdir('/build')
try:
if is_cmake_needed():
- cmake()
+ cmake(cmake_options)
ninja()
except CalledProcessError:
print("Build failed!")
@@ -55,8 +57,9 @@
"-DCLANG_ENABLE_STATIC_ANALYZER=ON"
-def cmake():
- check_call(CMAKE_COMMAND + ' /llvm-project/llvm', shell=True)
+def cmake(cmake_options):
+ check_call(CMAKE_COMMAND + ' '.join(cmake_options) + ' /llvm-project/llvm',
+ shell=True)
def ninja():
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105447.356821.patch
Type: text/x-patch
Size: 1394 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210706/5b534dec/attachment-0001.bin>
More information about the cfe-commits
mailing list