[PATCH] Make GDBJITRegistrar thread-safe

Kaylor, Andrew andrew.kaylor at intel.com
Thu Mar 13 10:45:06 PDT 2014


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<mailto: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<mailto: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/b880c74b/attachment.html>


More information about the llvm-commits mailing list