[PATCH] D52430: Introduce a flag to warn when IFUNC symbols are used with text relocations.

Ed Maste via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 26 07:15:49 PDT 2018


emaste added a comment.

> The benefit from this that we can track and update the status of the fix and even maybe provide info about the known distributions affected/fixed on the page what can be helpful for removing this flag at some longer-term point.

I prefer to have all documentation self-contained rather than relying on an external resource - we could put the information in the man page description for `--warn-ifunc-textrel`. As we already know when the glibc fix is available (upstream at least) we wouldn't have anything to keep up-to-date: we can simply describe the issue, the fixed glibc version, and note that the option will be removed in the future. Keeping track of which distros have which glibc versions seems like a lot of work.

That said, this issue doesn't affect the BSDs, so you ought to proceed with whatever is most natural/useful for Linux folks.



================
Comment at: docs/ld.lld.1:433
+Warn about usinc IFUNC symbols in conjunction with text relocations.
+This will cause segment faults if linked with glibc version 2.28 and before.
 .It Fl -warn-unresolved-symbols
----------------
segmentation faults


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D52430





More information about the llvm-commits mailing list