[Lldb-commits] [PATCH] build fixes for Linux

Malea, Daniel daniel.malea at intel.com
Wed Sep 5 16:31:27 PDT 2012

Thanks for taking a look Enrico. The PlatformRemoteiOS.cpp change just matches the brace from line 100 when __APPLE__ is not defined. However, on a closer inspection, it may be best to have the entire "default:" case outside the ifdef...


On 2012-09-05, at 6:40 PM, "Enrico Granata" <egranata at apple.com<mailto:egranata at apple.com>> wrote:

I am not sure about the PlatformRemoteiOS.cpp changes - but I have no objection to the other modifications brought in with this patch

Enrico Granata
✉ egranata@.com
✆ (408) 972-7683

On Sep 5, 2012, at 3:19 PM, "Malea, Daniel" <daniel.malea at intel.com<mailto:daniel.malea at intel.com>> wrote:


I noticed a few build problems on Linux introduced by recent commits. Please find the attached patch that addresses:
- missing include of stdint.h (required for int64_t)
- null return type in functions that must return SP
- off-by-one-line brace typo in PlatformRemoteiOS.cpp

With the patch applied (and the recently-comitted clang patch to enable the UTF functions) I am able to build trunk lldb on Linux :)


PS. git patch, so apply with -p1

lldb-commits mailing list
lldb-commits at cs.uiuc.edu<mailto:lldb-commits at cs.uiuc.edu>

More information about the lldb-commits mailing list