[Lldb-commits] [PATCH] [linux] Use cmake to detect support process_vm_readv (bug #23918)

Pavel Labath labath at google.com
Fri Jun 26 03:56:22 PDT 2015


My apologies, it seems I misunderstood the operation of check_cxx_source_compiles. Please try it again.


================
Comment at: cmake/modules/LLDBConfig.cmake:258
@@ +257,3 @@
+
+int main() { return process_vm_readv(0, 0, 0, 0, 0, 0); }
+"
----------------
ovyalov wrote:
> ovyalov wrote:
> > Do we need to cast from ssize_t to int?
> Could you pass nullptr for const struct iovec *?
done.

================
Comment at: source/Host/linux/LibcGlue.cpp:10
@@ -9,5 +9,3 @@
 
-// This files adds functions missing from libc on earlier versions of Android
-
-#include <android/api-level.h>
+// This files adds functions missing from libc on older versions of linux
 
----------------
ovyalov wrote:
> s/files/file
Done.

http://reviews.llvm.org/D10727

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list