[Openmp-commits] [PATCH] D118493: Set rpath on openmp executables

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

In D118493#3284617 <https://reviews.llvm.org/D118493#3284617>, @thakis wrote:

> looks like the mac linker doesn't like this test either: http://45.33.8.238/macm1/26834/step_7.txt

Error message is ld: file too small (length=8) file '/Users/thakis/src/llvm-project/clang/test/OpenMP/Inputs/libomp.a' for architecture arm64

On the face of it I'd say that's a bug in your linker. The archive file is the magic `!<arch>\n` string by itself which is well formed to the extent the archive format is documented. It's definitely possible to put more text in the stub file to work around though it's hard to tell how much text, or whether your linker will do other validation like seeing whether the symbol table is present or objects actually contain machine code.

I'm going to delete the test file in the meantime.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118493