[PATCH] D24356: ELF: accept -m elf_amd64 as an alias for -m elf_x86_64
Ed Maste via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 8 12:28:22 PDT 2016
emaste added inline comments.
================
Comment at: test/ELF/emulation.s:32-34
@@ -1,4 +31,5 @@
+
# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %tx64
# RUN: ld.lld -m elf_x86_64 %tx64 -o %t2x64
# RUN: llvm-readobj -file-headers %t2x64 | FileCheck --check-prefix=X86-64 %s
# RUN: ld.lld %tx64 -o %t3x64
----------------
ruiu wrote:
> Instead of adding "AMD64:" lines, can you reuse exiting "X86-64"? I think the output will be the same.
It's not completely identical -- note the OS/ABI. Right now I could probably change the `triple=x86_64-unknown-freebsd` to `-triple=x86_64-unknown-linux` and switch to `elf_amd64` from `elf_amd64_fbsd`, and then reuse the `X86-64` lines.
However, as the amd64 case is somewhat FreeBSD-specific I'd prefer to leave the FreeBSDisms in this test case and leave it as `elf_amd64_fbsd`.
https://reviews.llvm.org/D24356
More information about the llvm-commits
mailing list