<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Apr 8, 2016, 10:45 AM Paul Robinson <<a href="mailto:Paul_Robinson@playstation.sony.com">Paul_Robinson@playstation.sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">probinson added inline comments.<br>
<br>
================<br>
Comment at: include/clang/Basic/AttrDocs.td:2371<br>
@@ +2370,3 @@<br>
+<br>
+Not all targets support this attribute.  ELF targets support this attribute when using binutils v2.20.1 or higher and glibc v2.11.1 or higher.  Non-ELF targets currently do not support this attribute.<br>
+  }];<br>
----------------<br>
joerg wrote:<br>
> probinson wrote:<br>
> > echristo wrote:<br>
> > > rjmccall wrote:<br>
> > > > echristo wrote:<br>
> > > > > Probably better to say linux fwiw and not ELF.<br>
> > > > The validation code in Sema is checking for an ELF target.  If the restriction is more precise than that, then we should make a TargetInfo callback.  Do the BSDs and other ELF targets not use binutils/glibc?<br>
> > > We should make a TargetInfo callback. BSDs and other ELF targets aren't guaranteed to use binutils/glibc (some of them have even switched to llvm already) - and I don't know what the state of ifunc support on those architectures is.<br>
> > Hear hear. PS4 is ELF but we don't use glibc.<br>
> The attribute is not Linux specific, so ELF is a reasonable first approximation. Most BSDs have some ifunc support at least. I'm not in favor of doing any checks beyond ELF -- even on Linux the availability of working ifunc support depends on other factors like whether the binary is dynamically linked.<br>
What's the failure mode if the target doesn't actually support it?<br></blockquote></div><div><br></div><div>Unknown relocation likely. </div><div><br></div><div><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D15524" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15524</a><br>
<br>
<br>
<br>
</blockquote></div>