[PATCH] D9600: Add scan-build python implementation
Devin Coughlin via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 6 17:12:43 PST 2016
dcoughlin added a comment.
Hi Laszlo,
I've run scan-build-py with both environment-variable-based and library-based interposition on our open source benchmark suite and it looks like it is in great shape on Darwin! There are still some remaining issues with xcodebuild, but I will help fix these after scan-build-py is committed.
Please apply for commit access (the instructions are at http://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access), update the commit message, and commit.
Thanks for all your hard work on this!
================
Comment at: tools/scan-build-py/libscanbuild/intercept.py:261
@@ +260,3 @@
+
+ Same problem on linux when SELinux is enabled. The status query program
+ 'sestatus' and the output when it's enabled 'SELinux status: enabled'. """
----------------
Great! Thanks for checking for this.
================
Comment at: tools/scan-build-py/libscanbuild/runner.py:42
@@ +41,3 @@
+
+ at require(['command', 'directory', 'file', # an entry from compilation database
+ 'clang', 'direct_args', # compiler name, and arguments from command
----------------
Thanks for documenting these.
http://reviews.llvm.org/D9600
More information about the cfe-commits
mailing list