[all-commits] [llvm/llvm-project] 60a8a5: [llvm-objdump] Print file format in lowercase to m...

[Jordan Rupprecht via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 60a8a504f16dbbc5f2a6887ecb668ef4cb834949
      https://github.com/llvm/llvm-project/commit/60a8a504f16dbbc5f2a6887ecb668ef4cb834949
  Author: Jordan Rupprecht <rupprecht at google.com>
  Date:   2020-02-12 (Wed, 12 Feb 2020)

  Changed paths:
    M clang/test/Modules/pch_container.m
    M lld/test/COFF/savetemps.ll
    M llvm/test/CodeGen/AArch64/arm64-simplest-elf.ll
    M llvm/test/CodeGen/ARM/Windows/trivial-gnu-object.ll
    M llvm/test/CodeGen/BPF/reloc-btf-2.ll
    M llvm/test/CodeGen/BPF/reloc-btf.ll
    M llvm/test/CodeGen/BPF/reloc.ll
    M llvm/test/Object/AMDGPU/objdump.s
    M llvm/test/Object/X86/objdump-disassembly-inline-relocations.test
    M llvm/test/Object/X86/objdump-label.test
    M llvm/test/Object/X86/objdump-trivial-object.test
    M llvm/test/Object/dynamic-reloc.test
    M llvm/test/Object/objdump-symbol-table.test
    M llvm/test/tools/llvm-objdump/X86/disassemble-section-name.s
    M llvm/test/tools/llvm-objdump/X86/elf-disassemble-symbol-labels-exec.test
    M llvm/test/tools/llvm-objdump/X86/elf-dynamic-relocs.test
    M llvm/test/tools/llvm-objdump/X86/output-ordering.test
    M llvm/test/tools/llvm-objdump/X86/warn-missing-disasm-func.test
    M llvm/test/tools/llvm-objdump/all-headers.test
    M llvm/test/tools/llvm-objdump/archive-headers.test
    M llvm/test/tools/llvm-objdump/file-headers-coff.test
    M llvm/test/tools/llvm-objdump/file-headers-elf.test
    M llvm/test/tools/llvm-objdump/non-archive-object.test
    M llvm/test/tools/llvm-objdump/relocations-in-nonreloc.test
    M llvm/tools/llvm-objdump/llvm-objdump.cpp

  Log Message:
  -----------
  [llvm-objdump] Print file format in lowercase to match GNU output.

Summary:
GNU objdump prints the file format in lowercase, e.g. `elf64-x86-64`. llvm-objdump prints `ELF64-x86-64` right now, even though piping that into llvm-objcopy refuses that as a valid arch to use.

As an example of a problem this causes, see: https://github.com/ClangBuiltLinux/linux/issues/779

Reviewers: MaskRay, jhenderson, alexshap

Reviewed By: MaskRay

Subscribers: tpimh, sbc100, grimar, jvesely, nhaehnle, kerbowa, cfe-commits, llvm-commits

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D74433