[Lldb-commits] [PATCH] D17750: arm ios doesn't have elf.h, sys/procfs.h, need to avoid including them on apple systems

Jason Molenda via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 29 21:35:14 PST 2016

jasonmolenda created this revision.
jasonmolenda added reviewers: tberghammer, omjavaid.
jasonmolenda added a subscriber: lldb-commits.
jasonmolenda set the repository for this revision to rL LLVM.
Herald added subscribers: rengolin, aemerson.

I'm getting lldb to build for ios again and had build failures with NativeRegisterContextLinux_arm.cpp and Linux/Procfs.h which assume they're building for linux when being built or aarch64/etc.  

Would some patch like this be acceptable?  It's not pretty, but that's always the way with the C preprocessor as we pile on conditionals. :)




Index: source/Plugins/Process/Linux/Procfs.h
--- source/Plugins/Process/Linux/Procfs.h
+++ source/Plugins/Process/Linux/Procfs.h
@@ -26,6 +26,8 @@
 #endif // NT_FPREGSET
 #else  // __ANDROID__
+#if !defined (__APPLE__)
 #include <sys/procfs.h>
+#endif // __APPLE__
 #endif // __ANDROID__
Index: source/Plugins/Process/Linux/NativeRegisterContextLinux_arm.cpp
--- source/Plugins/Process/Linux/NativeRegisterContextLinux_arm.cpp
+++ source/Plugins/Process/Linux/NativeRegisterContextLinux_arm.cpp
@@ -7,7 +7,7 @@
-#if defined(__arm__) || defined(__arm64__) || defined(__aarch64__)
+#if !defined (__APPLE__) && (defined(__arm__) || defined(__arm64__) || defined(__aarch64__))
 #include "NativeRegisterContextLinux_arm.h"

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17750.49450.patch
Type: text/x-patch
Size: 956 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160301/4517c6dd/attachment.bin>

More information about the lldb-commits mailing list