[PATCH] D64943: [Clang][OpenMP offload] Eliminate use of OpenMP linker script

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added a comment.

In D64943#1666678 <https://reviews.llvm.org/D64943#1666678>, @JonChesterfield wrote:

> In D64943#1666673 <https://reviews.llvm.org/D64943#1666673>, @ABataev wrote:
>
> > In D64943#1666658 <https://reviews.llvm.org/D64943#1666658>, @JonChesterfield wrote:
> >
> > > > OpenMP linker script is known to cause problems for gold and lld linkers on Linux and it will also cause problems for Windows enabling in future
> > >
> > > What are the known problems with the linker script? I'm wondering if they can be resolved without the overhead of introducing a new tool.
> >
> >
> > They just do not support linker script. And, thus, cannot be used for offloading. Only `ld` supports it.
>
>
> In what respect? I've used linker scripts with both gold and lld, and both instances of --help text claim to support them. In the case of lld, a very complicated script hit a few internal errors, but I believe they've all been fixed since.


Hmm, I tried it with gold some time ago and it just did not work for me. The linking failed with diagnostics that some of the commands in the script are unknown.


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

https://reviews.llvm.org/D64943