[PATCH] D76046: [Object] Change ELFObjectFile<ELFT>::getFileFormatName() to use BFD names

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 03:10:00 PDT 2020


jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

Seems reasonable to me. LGTM.



================
Comment at: lld/test/ELF/basic-ppc.s:12
 
-// CHECK: Format: ELF32-ppc
+// CHECK: Format: elf32-powerpc
 // CHECK-NEXT: Arch: powerpc
----------------
Whilst you're modifying this line, would you mind lining it up with the line below?


================
Comment at: lld/test/ELF/basic-ppc64.s:17
 
-// CHECK:Format: ELF64-ppc64
+// CHECK:Format: elf64-powerpc
 // LE-NEXT:    Arch: powerpc64le
----------------
Ditto.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76046





More information about the llvm-commits mailing list