<div dir="ltr">Ahh yea I missed that.  Makes sense</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Feb 8, 2016 at 2:38 PM Siva Chandra <<a href="mailto:sivachandra@google.com">sivachandra@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Feb 8, 2016 at 2:35 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
><br>
><br>
> On Mon, Feb 8, 2016 at 2:21 PM Siva Chandra <<a href="mailto:sivachandra@google.com" target="_blank">sivachandra@google.com</a>> wrote:<br>
>><br>
>> On Mon, Feb 8, 2016 at 2:10 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
>> > Why though?<br>
>><br>
>> Foremost, I think it is because the lldb driver built as part of the<br>
>> test is linked to the host lldb shared library. If you build a 32-bit<br>
>> driver, we will need a 32-bit lldb shared library as well is it not?<br>
><br>
> yes, but the Makefile isn't specifying the architecture to build.  For me<br>
> it's building an x86 driver, my python distribution is x86 and the liblldb<br>
> shared library is x86 too.  So I don't see what the problem is.  If anything<br>
> I'm not sure why it's building an x64 driver on other platforms, since<br>
> nothing in the Makefile specifies that the Driver should be x64.<br>
><br>
>><br>
>><br>
>> Second, which I think can be fixed, is that we we load an x86_64<br>
>> target explicitly here: multi-process-driver.cpp:104<br>
><br>
> I missed that, and that is a good enough reason for now.  Although I think<br>
> it should be xfail instead of skip, because this is easily fixable by just<br>
> chekcing sizeof(void*) in the C++ side and passing a different arch when<br>
> creating the target.<br>
><br>
> As an aside, this seems like it would be much better as a unit test instead<br>
> of a python test, but I guess that's a different issue.<br>
><br>
> I'll change the skip to an xfail for now, but I still don't see anything<br>
> fundamentally incompatible with x86 here.<br>
<br>
That is why, in my first response, I said that a more apt skip<br>
decorator would be @skipIfNotHostArch (after ofcourse fixing the<br>
explicit x86_64 target requirement in the driver.)<br>
</blockquote></div>