<div dir="ltr">Hi Ivan, Jim,<div><br></div><div>Sorry - this one slipped by me. I've committed it in r203853. Thanks very much for the patch!</div><div><br></div><div>Cheers,</div><div>Lang.</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Mar 13, 2014 at 1:03 PM, Ivan Puzyrevskiy <span dir="ltr"><<a href="mailto:sandello@yandex-team.ru" target="_blank">sandello@yandex-team.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The one attached to the first email in this thread.<br>
Reattaching it for convenience.<br>
<br>
Best,<br>
Ivan<br>
<br><br>
On 13 марта 2014 г., at 22:15, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
<br>
> I'll get it. Which patches need to be applied?<br>
><br>
> -eric<br>
><br>
> On Thu, Mar 13, 2014 at 10:57 AM, Kaylor, Andrew<br>
> <<a href="mailto:andrew.kaylor@intel.com">andrew.kaylor@intel.com</a>> wrote:<br>
>> Hi Ivan,<br>
>><br>
>><br>
>><br>
>> I'm not properly setup to commit this at the moment.  I was hoping either<br>
>> Tim or Lang would be willing to commit it.<br>
>><br>
>><br>
>><br>
>> -Andy<br>
>><br>
>><br>
>><br>
>> From: Ivan Puzyrevskiy [mailto:<a href="mailto:sandello@yandex-team.ru">sandello@yandex-team.ru</a>]<br>
>> Sent: Thursday, March 13, 2014 10:49 AM<br>
>> To: Kaylor, Andrew<br>
>> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>; <a href="mailto:jim.kearney@oracle.com">jim.kearney@oracle.com</a>; Lang Hames<br>
>> (<a href="mailto:lhames@apple.com">lhames@apple.com</a>); Tim Northover<br>
>> Subject: Re: [PATCH] Make GDBJITRegistrar thread-safe<br>
>><br>
>><br>
>><br>
>> Jim pointed out to me that patch was approved some time ago but it didn't<br>
>> make its way to the trunk. See:<br>
>><br>
>><br>
>><br>
>> <a href="http://comments.gmane.org/gmane.comp.compilers.llvm.cvs/161966" target="_blank">http://comments.gmane.org/gmane.comp.compilers.llvm.cvs/161966</a><br>
>> <a href="http://comments.gmane.org/gmane.comp.compilers.llvm.cvs/161965" target="_blank">http://comments.gmane.org/gmane.comp.compilers.llvm.cvs/161965</a><br>
>><br>
>><br>
>><br>
>> Andy, if this patch would be approved can I ask you to commit it? Neither me<br>
>> nor Jim has commit rights.<br>
>><br>
>><br>
>><br>
>> Best,<br>
>><br>
>> Ivan.<br>
>><br>
>><br>
>><br>
>> On 13 марта 2014 г., at 21:45, Kaylor, Andrew <<a href="mailto:andrew.kaylor@intel.com">andrew.kaylor@intel.com</a>><br>
>> wrote:<br>
>><br>
>><br>
>><br>
>> That looks good as an intermediate fix.<br>
>><br>
>><br>
>><br>
>> In the long term I think the GDB registration should probably be moved out<br>
>> of RuntimeDyldELF (maybe to an event listener), but these changes would<br>
>> probably be nice to have even then.<br>
>><br>
>><br>
>><br>
>> -Andy<br>
>><br>
>><br>
>><br>
>> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a><br>
>> [mailto:<a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Ivan Puzyrevskiy<br>
>> Sent: Thursday, March 13, 2014 4:45 AM<br>
>> To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> Cc: <a href="mailto:jim.kearney@oracle.com">jim.kearney@oracle.com</a><br>
>> Subject: [PATCH] Make GDBJITRegistrar thread-safe<br>
>><br>
>><br>
>><br>
>> Hi there.<br>
>><br>
>> While investigating a crash in GDBJITRegistrar I have stumbled upon<br>
>> <a href="http://llvm.org/bugs/show_bug.cgi?id=15750" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=15750</a> . In my setting there were<br>
>> multiple instances of MCJIT operating in different threads.<br>
>><br>
>> Patch from Bugzilla applies cleanly and solves the issue. I have attached a<br>
>> patch against r203780, yet full credit goes to Jim Kearney<br>
>> (<a href="mailto:jim.kearney@oracle.com">jim.kearney@oracle.com</a>), original author of the patch.<br>
>><br>
>> Any chances to see this in the trunk or 3.5 release?<br>
>><br>
>> Best,<br>
>> Ivan<br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>><br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>><br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>