[Lldb-commits] [PATCH] Make llgs build on Android. No functionality changes.

jingham at apple.com jingham at apple.com
Thu Sep 25 16:23:04 PDT 2014


I don't have time to look at this right now, but that's an awful lot of #ifndef __ANDROID__ in places I wouldn't really expect these defines.  Particularly the ones that deal with the IOHandlerEditLine.  Either you're building stuff for llgs that you don't really need to (why does it need the command interpreter) or we really need a non-interactive configuration for the command interpreter that substitutes the EditLine I/O handler with one that is not interactive...  Anyway, that part does seem ugly to me.

Also one of the uses of __ANDROID__ is to not include some python goo, but there's already a "build without python" define for that purpose.

Jim


> On Sep 25, 2014, at 4:15 PM, Todd Fiala <tfiala at google.com> wrote:
> 
> LGTM.
> 
> We might later want to look at how/where we're constructing IOHandlerEditLine.  It seems like it might first show up at too low a level of the stack based on your need to comment it out (unneeded in llgs and not wanting to bring in the libedit dependency).
> 
> I'll ping Greg to see if he wants somebody to look over before we approve and move this in.
> 
> http://reviews.llvm.org/D5495
> 
> 
> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits




More information about the lldb-commits mailing list