<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Feb 8, 2016 at 2:21 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: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></blockquote><div>yes, but the Makefile isn't specifying the architecture to build. For me it's building an x86 driver, my python distribution is x86 and the liblldb shared library is x86 too. So I don't see what the problem is. If anything I'm not sure why it's building an x64 driver on other platforms, since nothing in the Makefile specifies that the Driver should be x64. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<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></blockquote><div>I missed that, and that is a good enough reason for now. Although I think it should be xfail instead of skip, because this is easily fixable by just chekcing sizeof(void*) in the C++ side and passing a different arch when creating the target.</div><div><br></div><div>As an aside, this seems like it would be much better as a unit test instead of a python test, but I guess that's a different issue.</div><div><br></div><div>I'll change the skip to an xfail for now, but I still don't see anything fundamentally incompatible with x86 here. </div></div></div>