<div dir="ltr">Hi Anthony,<div><br></div><div>I just wanted to double check - did you ever end up re-committing any aspect of this patch? (It doesn't look like it, but I thought I'd double check with you). I'm about to take another look at this and I want to make sure I don't duplicate any work.</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 23, 2014 at 3:29 PM, Anthony Pesch <span dir="ltr"><<a href="mailto:inolen@gmail.com" target="_blank">inolen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I do, I'll give it a look tonight.<br>
<br>
Thanks!<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Oct 23, 2014 at 1:53 PM, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:<br>
> Hi Anthony,<br>
><br>
> It looks like this reproduces on x86-64 Linux. Do you have access to a Linux<br>
> box?<br>
><br>
> Cheers,<br>
> Lang.<br>
><br>
> On Wed, Oct 22, 2014 at 11:54 AM, Anthony Pesch <<a href="mailto:inolen@gmail.com">inolen@gmail.com</a>> wrote:<br>
>><br>
>> Is there anything I can do to help / reproduce the issue locally? I'd<br>
>> ran `make check` on my Mac and it seemed well from that perspective.<br>
>><br>
>>  - Anthony<br>
>><br>
>> On Mon, Oct 20, 2014 at 5:29 PM, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:<br>
>> > Unfortunately this broke some of the bots. I'm going to temporarily<br>
>> > revert<br>
>> > it while I try to track down the issue.<br>
>> ><br>
>> > Cheers,<br>
>> > Lang.<br>
>> ><br>
>> > On Mon, Oct 20, 2014 at 4:52 PM, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:<br>
>> >><br>
>> >> Looks good to me. Committed in r220245. Thanks very much for all your<br>
>> >> work<br>
>> >> on this Anthony!<br>
>> >><br>
>> >> Thanks for the help with the review too Rafael.<br>
>> >><br>
>> >> Cheers,<br>
>> >> Lang.<br>
>> >><br>
>> >> On Mon, Oct 20, 2014 at 11:17 AM, Rafael Espíndola<br>
>> >> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> >>><br>
>> >>> This patch looks good to me.<br>
>> >>><br>
>> >>> Lang, can you commit if the MCJIT bits look good to you?<br>
>> >>><br>
>> >>> On 20 October 2014 12:28, Anthony Pesch <<a href="mailto:inolen@gmail.com">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<br>
>> >>> > 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">rafael.espindola@gmail.com</a>> wrote:<br>
>> >>> >> On 19 October 2014 18:16, Anthony Pesch <<a href="mailto:inolen@gmail.com">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<br>
>> >>> >> of<br>
>> >>> >> the file. Do you have commit access?<br>
>> >>> >><br>
>> >>> >> Cheers,<br>
>> >>> >> Rafael<br>
>> >><br>
>> >><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div>