[PATCH] D40849: CodeGen: support an extension to pass linker options on ELF

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 20 15:43:28 PST 2017


probinson added a comment.

In https://reviews.llvm.org/D40849#961429, @silvas wrote:

> - I know the PlayStation proprietary linker has support for a similar feature (as James has been discussing) and IIRC there were private patches in the compiler for it. Is it in scope for this change to allow deleting those private patches from the PlayStation compiler?


James would be more in tune with what the PS4 linker supports than I am, but in the compiler we don't do arbitrary linker options.  We support specifying libraries (a-la `pragma comment lib`) and dllimport/dllexport names.  The proposed patch could be used for the former but I think not the latter.  It's also not clear whether our mechanism for dllimport/dllexport is optimal, but it keeps us away from all the angst about trying to pervert ELF visibility into supporting it.


Repository:
  rL LLVM

https://reviews.llvm.org/D40849





More information about the llvm-commits mailing list