<div dir="ltr">Uh, yeah, that looks racy. I'd suggest guarding it in *SignalsMutex. I'd recommend filing a bug or sending a patch.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 2, 2016 at 1:03 AM, Viacheslav Nikolaev via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>There was an issue reported a while ago</div><a href="http://lists.llvm.org/pipermail/llvm-dev/2014-February/070692.html" target="_blank">http://lists.llvm.org/<wbr>pipermail/llvm-dev/2014-<wbr>February/070692.html</a><br><div><br></div><div>but it is still not adressed. RegisterHandlers() is not thread safe and isn't called under locks for Unix...</div><div><br></div><div>Is there any reason why RegisterHandlers() shouldn't be put under the scoped locks used before the invocations of the function?</div></div>
<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>