[LLVMdev] Usage of getenv() inside LLVM and thread safety
silvas at purdue.edu
Thu May 23 11:30:22 PDT 2013
On Thu, May 23, 2013 at 8:13 AM, Justin Holewinski <
justin.holewinski at gmail.com> wrote:
> That sounds like a missed multi-threading issue with LLVM. I can't
> imagine why the user should be forced to serialize creation of MCContext
> objects. I would suggest filing a bug for this. A simple lock probably
> wouldn't be too detrimental to performance here, since MCContext objects
> shouldn't be created too often.
> The deeper question is why we are even checking a "global" here in the
first place? It goes against LLVM's library-based design. So I don't think
introducing locking around this is the "right" solution. Can we just make
this value an argument to the constructor?
-- Sean Silva
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev