<div dir="ltr">Unfortunately this broke some of the bots. I'm going to temporarily revert it while I try to track down the issue.<div><br></div><div>Cheers,</div><div>Lang.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 20, 2014 at 4:52 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Looks good to me. Committed in r220245. Thanks very much for all your work on this Anthony!<div><br></div><div>Thanks for the help with the review too Rafael.</div><div><br></div><div>Cheers,</div><div>Lang.</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 20, 2014 at 11:17 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch looks good to me.<br>
<br>
Lang, can you commit if the MCJIT bits look good to you?<br>
<div><div><br>
On 20 October 2014 12:28, Anthony Pesch <<a href="mailto:inolen@gmail.com" target="_blank">inolen@gmail.com</a>> wrote:<br>
> I don't have commit access. Should I get that setup or can someone<br>
> commit on my behalf?<br>
><br>
> I've attached a patch without any spacing changes (sorry for the noise).<br>
><br>
>  - Anthony<br>
><br>
> On Mon, Oct 20, 2014 at 7:29 AM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br>
>> On 19 October 2014 18:16, Anthony Pesch <<a href="mailto:inolen@gmail.com" target="_blank">inolen@gmail.com</a>> wrote:<br>
>>> Hey guys,<br>
>>><br>
>>> Alright, updated the patch, added isExportedToOtherDSO, updated<br>
>>> comments, and converted the tests to spaces instead of tabs.<br>
>><br>
>> The ELF bits looks good.<br>
>><br>
>> Please commit the tabs to spaces change by itself before the rest of<br>
>> the file. Do you have commit access?<br>
>><br>
>> Cheers,<br>
>> Rafael<br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>