joerg added a comment.
Yeah, I would restrict it to just mention that it depends on the target, link
time editor and runtime linker. Even the concrete feature set on Linux changes
with glibc versions.
Repository:
rL LLVM
https://reviews.llvm.org/D52696
___
emaste added a comment.
Maybe "available for some architectures in at least..."? Or maybe we shouldn't
bother trying to list versions, and mention it is dependent on CPU arch,
linker, and rtld?
Repository:
rL LLVM
https://reviews.llvm.org/D52696
__
joerg added a comment.
I think this is still too optimistic. Full support for ifunc seems to be
generally limited to x86. Most other architectures lack even definitions for
anonymous ifunc relocations or support proper relaxation only in limited forms.
That's especially annoying when looking at
This revision was automatically updated to reflect the committed changes.
Closed by commit rL343408: Update ifunc attribute support documentation
(authored by emaste, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D52696?vs=167626&id=1
DmitryPolukhin accepted this revision.
DmitryPolukhin added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D52696
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
emaste created this revision.
emaste added a reviewer: DmitryPolukhin.
Herald added a subscriber: krytarowski.
We documented GNU binutils and glibc versions required for ifunc support, but
our own lld linker and FreeBSD's rtld also support ifuncs.
https://reviews.llvm.org/D52696
Files:
inclu
