[PATCH] D156468: [lld-macho] Fixed crashes when linking with incompatible-arch archives/

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lld/test/MachO/ignore-incompat-arch.s:70
+_main:
+  .quad _caller
+  ret
----------------
oontvoo wrote:
> MaskRay wrote:
> > it'd be slightly better to use a call instead of a data relocation.
> Originally I'd used the normal `call[q]` but llvm-mc choked when compiling for arm64 and I didn't want to have two separate input for x86 vs arm64.
> So I'd figured `.quad` is more portable - acceptable for both archs
> 
> Can you suggest an alternative?
Ah, ok, for portability, `.quad` looks good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156468