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

2016-03-01 Thread Jason Molenda via lldb-commits
jasonmolenda abandoned this revision. jasonmolenda added a comment. Yes, taking the source/Plugins/Process/Linux directory out of the xcodeproject file altogether makes this build on ios, I'll go with that approach. Repository: rL LLVM http://reviews.llvm.org/D17750 __

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

2016-03-01 Thread Jason Molenda via lldb-commits
jasonmolenda added a comment. Good point. Let me play with the xcode project file (which I was using to do the build) tomorrow; your solution may be the correct one, we might have some files being built unnecessarily. Repository: rL LLVM http://reviews.llvm.org/D17750 __

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

2016-03-01 Thread Pavel Labath via lldb-commits
labath added a subscriber: labath. labath requested changes to this revision. labath added a reviewer: labath. labath added a comment. This revision now requires changes to proceed. The actual changes aren't too bad, but I wonder why you need to add them at all. These files all assume they will b

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

2016-02-29 Thread Jason Molenda via lldb-commits
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