Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2395)

Unified Diff: build/toolchain/clang_static_analyzer_wrapper.py

Issue 2617283002: Add Clang static analyzer to Clang toolchain defs in GN (Closed)
Patch Set: wez feedback Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/toolchain/clang_static_analyzer_wrapper.py
diff --git a/build/toolchain/clang_static_analyzer_wrapper.py b/build/toolchain/clang_static_analyzer_wrapper.py
new file mode 100755
index 0000000000000000000000000000000000000000..68258ff74ef446114454a06f5622d77feb046ca7
--- /dev/null
+++ b/build/toolchain/clang_static_analyzer_wrapper.py
@@ -0,0 +1,60 @@
+#!/usr/bin/env python
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""Invokes the Clang static analysis command using arguments provided on the
+command line.
+"""
+
+import argparse
+import fnmatch
+import os
+import shutil
+import sys
+import tempfile
+
+import wrapper_utils
+
+
+def main():
+ parser = argparse.ArgumentParser(description=__doc__)
+ parser.add_argument('--clang-cc-path',
+ help='Path to the clang compiler.',
+ metavar='PATH')
+ parser.add_argument('--clang-cxx-path',
+ help='Path to the clang++ compiler',
+ metavar='PATH')
+ parser.add_argument('--analyzer',
+ help='Path to the language-specific Clang analysis tool.',
+ required=True,
+ metavar='PATH')
+ args, compile_args = parser.parse_known_args()
+
+ # Check that only one of --clang-cc-path or --clang-cxx-path are set.
+ assert ((args.clang_cc_path != None) != (args.clang_cxx_path != None))
+
+ is_cxx = args.clang_cxx_path != None
+ env = os.environ
+ env['CCC_ANALYZER_FORCE_ANALYZE_DEBUG_CODE'] = '0'
+ env['CCC_ANALYZER_OUTPUT_FORMAT'] = 'text'
+ clang_path = args.clang_cxx_path or args.clang_cc_path
+ if is_cxx:
+ env['CCC_CXX'] = clang_path
+ env['CLANG_CXX'] = clang_path
+ else:
+ env['CCC_CC'] = clang_path
+ env['CLANG'] = clang_path
+
+ # TODO(kmarshall): Place the summarized output in a useful directory.
+ temp_dir = tempfile.mkdtemp()
+ try:
+ env['CCC_ANALYZER_HTML'] = temp_dir
+ returncode = wrapper_utils.RunCommand(
+ wrapper_utils.CommandToRun([args.analyzer] + compile_args), env)
+ return returncode
+ finally:
+ shutil.rmtree(temp_dir)
+
+if __name__ == "__main__":
+ sys.exit(main())

Powered by Google App Engine
This is Rietveld 408576698