[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