[PATCH] Don't build DFSan functionality for Android.

Stephen Hines srhines at google.com
Thu Feb 13 13:31:55 PST 2014


Submitted in r201362.

Thanks,
Steve


On Thu, Feb 13, 2014 at 11:13 AM, Peter Collingbourne <peter at pcc.me.uk>wrote:

> On Thu, Feb 13, 2014 at 01:14:02AM -0800, Stephen Hines wrote:
> > Hi Peter,
> >
> > We don't include DFSan as part of compiler-rt on Android, so these extern
> > TLS declarations are not available (and thus result in link errors for
> LLVM
> > 3.4). Let me know if this is ok, or if there is some other way you think
> we
> > can resolve this.
> >
> > Thanks,
> > Steve
>
> I'm surprised that the compiler emits a reference to the TLS symbols
> because
> the function is normally unreferenced. But this looks like the right thing
> to
> do on Android because __thread doesn't seem to be supported on that
> platform,
> so LGTM.
>
> Thanks,
> --
> Peter
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140213/6f736744/attachment.html>


More information about the llvm-commits mailing list