thakis added a comment. In D118493#3284663 <https://reviews.llvm.org/D118493#3284663>, @JonChesterfield wrote:
> 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. It's just the standard Xcode linker, I didn't write it :) > 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. > edit: or less aggressive, is there an UNSUPPORTED flag I can add that will > drop it from your CI? Maybe you can pass an explicit `--target` triple to clang, and just check `-###` instead of actually linking, like driver tests usually do? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118493/new/ https://reviews.llvm.org/D118493 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
