[PATCH] Make GDBJITRegistrar thread-safe

[Ivan Puzyrevskiy]

Jim pointed out to me that patch was approved some time ago but it didn’t make its way to the trunk. See:

http://comments.gmane.org/gmane.comp.compilers.llvm.cvs/161966
http://comments.gmane.org/gmane.comp.compilers.llvm.cvs/161965

Andy, if this patch would be approved can I ask you to commit it? Neither me nor Jim has commit rights.

Best,
Ivan.

On 13 марта 2014 г., at 21:45, Kaylor, Andrew <andrew.kaylor at intel.com> wrote:

> That looks good as an intermediate fix.
>  
> In the long term I think the GDB registration should probably be moved out of RuntimeDyldELF (maybe to an event listener), but these changes would probably be nice to have even then.
>  
> -Andy
>  
> From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-bounces at cs.uiuc.edu] On Behalf Of Ivan Puzyrevskiy
> Sent: Thursday, March 13, 2014 4:45 AM
> To: llvm-commits at cs.uiuc.edu
> Cc: jim.kearney at oracle.com
> Subject: [PATCH] Make GDBJITRegistrar thread-safe
>  
> Hi there.
> 
> While investigating a crash in GDBJITRegistrar I have stumbled upon http://llvm.org/bugs/show_bug.cgi?id=15750 . In my setting there were multiple instances of MCJIT operating in different threads.
> 
> Patch from Bugzilla applies cleanly and solves the issue. I have attached a patch against r203780, yet full credit goes to Jim Kearney (jim.kearney at oracle.com), original author of the patch.
> 
> Any chances to see this in the trunk or 3.5 release?
> 
> Best, 
> Ivan
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140313/11bfd452/attachment.html>