xazax.hun added inline comments.

================
Comment at: utils/analyzer/SATestBuild.py:277
@@ +276,3 @@
+    # For now, we assume the preprocessed files should be analyzed
+    # with the OS X SDK.
+    SDKPath = getSDKPath("macosx")
----------------
dcoughlin wrote:
> xazax.hun wrote:
> > I think it might be better to check if the host os is OS X. This way one 
> > might be able to check preprocessed files on other linux.
> The patch checks for the existence of xcrun in the path and doesn't set 
> -isysroot if that is not present, so this should continue to work on Linux 
> and cygwin.
I see, however -cc1 option is only used when the SDKPath is not None. Is this 
intended?


http://reviews.llvm.org/D12769



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to