diff options
| author | Jordan Rupprecht <rupprecht@google.com> | 2020-02-11 11:55:40 -0800 |
|---|---|---|
| committer | Jordan Rupprecht <rupprecht@google.com> | 2020-02-12 08:17:01 -0800 |
| commit | 60a8a504f16dbbc5f2a6887ecb668ef4cb834949 (patch) | |
| tree | f357b0b9290e2d5fd6cc98bea95cd73ce781d7de /llvm/tools/llvm-objdump/llvm-objdump.cpp | |
| parent | ff307c8120ecca89193576c38d68a0353ea49e8b (diff) | |
| download | llvm-60a8a504f16dbbc5f2a6887ecb668ef4cb834949.zip llvm-60a8a504f16dbbc5f2a6887ecb668ef4cb834949.tar.gz llvm-60a8a504f16dbbc5f2a6887ecb668ef4cb834949.tar.bz2 | |
[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
Diffstat (limited to 'llvm/tools/llvm-objdump/llvm-objdump.cpp')
| -rw-r--r-- | llvm/tools/llvm-objdump/llvm-objdump.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/tools/llvm-objdump/llvm-objdump.cpp b/llvm/tools/llvm-objdump/llvm-objdump.cpp index a82fbb2..1b98d78 100644 --- a/llvm/tools/llvm-objdump/llvm-objdump.cpp +++ b/llvm/tools/llvm-objdump/llvm-objdump.cpp @@ -2168,7 +2168,7 @@ static void dumpObject(ObjectFile *O, const Archive *A = nullptr, outs() << A->getFileName() << "(" << O->getFileName() << ")"; else outs() << O->getFileName(); - outs() << ":\tfile format " << O->getFileFormatName() << "\n\n"; + outs() << ":\tfile format " << O->getFileFormatName().lower() << "\n\n"; } if (StartAddress.getNumOccurrences() || StopAddress.getNumOccurrences()) |