<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Submitted in r201362.</span><br><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Thanks,</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px">Steve</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 13, 2014 at 11:13 AM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Feb 13, 2014 at 01:14:02AM -0800, Stephen Hines wrote:<br>
> Hi Peter,<br>
><br>
> We don't include DFSan as part of compiler-rt on Android, so these extern<br>
> TLS declarations are not available (and thus result in link errors for LLVM<br>
> 3.4). Let me know if this is ok, or if there is some other way you think we<br>
> can resolve this.<br>
><br>
> Thanks,<br>
> Steve<br>
<br>
</div></div>I'm surprised that the compiler emits a reference to the TLS symbols because<br>
the function is normally unreferenced. But this looks like the right thing to<br>
do on Android because __thread doesn't seem to be supported on that platform,<br>
so LGTM.<br>
<br>
Thanks,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Peter<br>
</font></span></blockquote></div><br></div>