Hey Todd,

Yes, you are correct. I needed this patch for non-Apple targets that send a 
cputype/cpusubtype couple instead of a full triple. If this is something that 
you guys think is invalid, I can deal with it on the debug server we are using.

AFAICT, right now we have two ways of getting the process info: one with the 
triples, which works everywhere, and one with cputype/cpusubtype that works 
only with apple targets. This would make both ways work everywhere in theory.

On Sep 10, 2014, at 10:36 AM, Todd Fiala <tfi...@google.com> wrote:

> Hey Stephane!
> 
> On this patch, one of the things I'm seeing is that it appears you are 
> (maybe?) sending cputype and cpusubtype in cases where the target is not a 
> MachO-based system.  In general, the cputype/cpusubtype are meant to be a 
> MachO-xnu specific mechanism.  We try to *not* send those for non-Apple 
> targets and instead send just the triples.
> 
> Are you in a position where this might be the case?  I can try out the patch 
> but it looks like it's basically geared to handle qProcessInfo for non-MachO 
> platforms sending cpu type info if I'm reading it right.
> 
> -Todd
> 
> On Tue, Sep 9, 2014 at 5:23 PM, Stephane Sezer <s...@fb.com> wrote:
> Instead of forcing the remote arch type to MachO all the time, we
> inspect the OS/vendor that the remote debug server reports and use it to
> set the arch type to MachO, ELF or COFF accordingly.
> 
> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits@cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
> 
> 
> 
> 
> -- 
> Todd Fiala |   Software Engineer |     tfi...@google.com |     650-943-3180
> 


_______________________________________________
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to