[PATCH] D113443: [FreeBSD] avoid marking __stack_chk_guard symbol as dso_local on PPC64

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 22 14:35:47 PST 2021


MaskRay added a comment.

`readelf -r` without `-W` may not show the full relocation type name.

The relocation type is `R_X86_64_REX_GOTPCRELX`, not `R_X86_64_REX_GOTP`.

As I mentioned, `-fno-pic` may need GOT-generating relocations as well to avoid copy relocations. Not handling `R_X86_64_REX_GOTPCRELX` is FreeBSD's limitation which should be fixed https://maskray.me/blog/2021-08-29-all-about-global-offset-table

I haven't read all the PowerPC discussions but it seems that there may be a dso_local PowerPC64 ELFv2 issue as well which should be properly fixed.

With all these said, I am fine that this is pushed to fix the immediate problem for FreeBSD. But if FreeBSD 14.0.0 plans to fix `R_X86_64_REX_GOTPCRELX`, this patch is probably not very necessary.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113443/new/

https://reviews.llvm.org/D113443



More information about the llvm-commits mailing list