<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 30, 2014 at 10:35 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</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>llvm-dev -> cfe-dev, since ASTUnit.h is a Clang thing.</div><div><br></div><div>Maybe Manuel knows what it is, since it's used by Tooling?</div></div></blockquote><div><br></div><div>I'm not aware that ASTUnit is used by libTooling. As far as I know, clang is not intended to be thread-safe, and thus probably isn't.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">
On Fri, May 30, 2014 at 12:56 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5">
<div dir="ltr">In ASTUnit.h, there is a ConcurrencyState class which has the following comment;<div><br></div><div><div>  /// \brief Allows us to assert that ASTUnit is not being used concurrently,</div><div>  /// which is not supported.</div>


<div>  ///</div><div>  /// Clients should create instances of the ConcurrencyCheck class whenever</div><div>  /// using the ASTUnit in a way that isn't intended to be concurrent, which is</div><div>  /// just about any usage.</div>


<div>  /// Becomes a noop in release mode; only useful for debug mode checking.</div></div><div><br></div><div>In ASTUnit.cpp, there is a function called cleanupOnDiskMapAtExit(), which has the following comment:</div><div>


<br></div><div><div>  // Use the mutex because there can be an alive thread destroying an ASTUnit.</div></div><div><br></div><div>Can someone clarify the intended behavior here?  These two comments seem to contradict each other.</div>


</div>
<br></div></div>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div></div>