[PATCH] D26133: [ELF] Allow relative relocations to absolute symbols in PIC

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 30 22:24:39 PDT 2016


pcc added a subscriber: rafael.
pcc added a comment.

I don't see why we should switch from rejecting to doing the wrong thing. The right thing to do, as Ian suggested, would be to create either a PLT entry or a dynamic relocation.

Independently of that, as I and Rafael mentioned on the review thread for https://reviews.llvm.org/D25560, we need to fix the synthetic symbols we create for linker scripts.


Repository:
  rL LLVM

https://reviews.llvm.org/D26133





More information about the llvm-commits mailing list