[PATCH] D55505: [LLD][ELF] Implement option to force PIC compatible Thunks

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 7 03:19:20 PST 2019


peter.smith added a comment.

In D55505#1343979 <https://reviews.llvm.org/D55505#1343979>, @ruiu wrote:

> Generally looking good.
>
> What is a reason to add the option as `--pic-thunk` with an alias of `--pic-veneer`? If existing code uses `--pic-veneer`, maybe we could just add that option?


It was only to be consistent with LLD's use of thunk. I'll update the patch with just --pic-veneer as the command line option to select it.


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

https://reviews.llvm.org/D55505





More information about the llvm-commits mailing list