<div><br></div><div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 10, 2018 at 6:38 PM Hui Huang via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hui added inline comments.<br>
<br>
<br>
================<br>
Comment at: source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp:810<br>
+<br>
+  std::lock_guard<std::recursive_mutex> guard(module_sp->GetMutex());<br>
+<br>
----------------<br>
zturner wrote:<br>
> Does this actually need to be a `recursive_mutex`?<br>
I think there is access to the member 'm_filespec_</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ap'.</blockquote><div dir="auto"><br></div><div dir="auto">I mean instead of a non recursive mutex.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div>