[PATCH] D70570: [PowerPC] Only use PLT annotations if using PIC relocation model

Vitaly Cheptsov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 29 06:03:03 PST 2019


vit9696 added a reviewer: vit9696.
vit9696 added a comment.

This change makes good sense to me, as I suggested pretty much the same thing 2 years ago (D38554 <https://reviews.llvm.org/D38554>). In the end for us it was easier to fix LLD and switch to GNU ABI ignoring IFUNC support entirely than to convince the others and support the change. Yet either way believe that it is logical not to emit PLT for static memory model. Personally I can accept this, but I wonder whether there are more comments coming.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70570





More information about the llvm-commits mailing list