diff options
Diffstat (limited to 'llvm/tools')
-rw-r--r-- | llvm/tools/llvm-readobj/ELFDumper.cpp | 49 | ||||
-rw-r--r-- | llvm/tools/llvm-readobj/ObjDumper.cpp | 13 | ||||
-rw-r--r-- | llvm/tools/llvm-readobj/ObjDumper.h | 4 | ||||
-rw-r--r-- | llvm/tools/llvm-readobj/Opts.td | 3 | ||||
-rw-r--r-- | llvm/tools/llvm-readobj/llvm-readobj.cpp | 49 | ||||
-rw-r--r-- | llvm/tools/llvm-readobj/llvm-readobj.h | 2 |
6 files changed, 25 insertions, 95 deletions
diff --git a/llvm/tools/llvm-readobj/ELFDumper.cpp b/llvm/tools/llvm-readobj/ELFDumper.cpp index 2f64b07..431b20f 100644 --- a/llvm/tools/llvm-readobj/ELFDumper.cpp +++ b/llvm/tools/llvm-readobj/ELFDumper.cpp @@ -31,7 +31,6 @@ #include "llvm/BinaryFormat/AMDGPUMetadataVerifier.h" #include "llvm/BinaryFormat/ELF.h" #include "llvm/Demangle/Demangle.h" -#include "llvm/Object/Archive.h" #include "llvm/Object/ELF.h" #include "llvm/Object/ELFObjectFile.h" #include "llvm/Object/ELFTypes.h" @@ -549,9 +548,6 @@ public: assert(&this->W.getOStream() == &llvm::fouts()); } - void printFileSummary(StringRef FileStr, ObjectFile &Obj, - ArrayRef<std::string> InputFilenames, - const Archive *A) override; void printFileHeaders() override; void printGroupSections() override; void printRelocations() override; @@ -701,27 +697,9 @@ private: void printMipsPLT(const MipsGOTParser<ELFT> &Parser) override; void printMipsABIFlags() override; -protected: ScopedPrinter &W; }; -// JSONELFDumper shares most of the same implementation as LLVMELFDumper except -// it uses a JSONScopedPrinter. -template <typename ELFT> class JSONELFDumper : public LLVMELFDumper<ELFT> { -public: - LLVM_ELF_IMPORT_TYPES_ELFT(ELFT) - - JSONELFDumper(const object::ELFObjectFile<ELFT> &ObjF, ScopedPrinter &Writer) - : LLVMELFDumper<ELFT>(ObjF, Writer) {} - - void printFileSummary(StringRef FileStr, ObjectFile &Obj, - ArrayRef<std::string> InputFilenames, - const Archive *A) override; - -private: - std::unique_ptr<DictScope> FileScope; -}; - } // end anonymous namespace namespace llvm { @@ -731,8 +709,6 @@ static std::unique_ptr<ObjDumper> createELFDumper(const ELFObjectFile<ELFT> &Obj, ScopedPrinter &Writer) { if (opts::Output == opts::GNU) return std::make_unique<GNUELFDumper<ELFT>>(Obj, Writer); - else if (opts::Output == opts::JSON) - return std::make_unique<JSONELFDumper<ELFT>>(Obj, Writer); return std::make_unique<LLVMELFDumper<ELFT>>(Obj, Writer); } @@ -3249,16 +3225,6 @@ static const EnumEntry<unsigned> *getObjectFileEnumEntry(unsigned Type) { return nullptr; } -template <class ELFT> -void GNUELFDumper<ELFT>::printFileSummary(StringRef FileStr, ObjectFile &Obj, - ArrayRef<std::string> InputFilenames, - const Archive *A) { - if (InputFilenames.size() > 1 || A) { - this->W.startLine() << "\n"; - this->W.printString("File", FileStr); - } -} - template <class ELFT> void GNUELFDumper<ELFT>::printFileHeaders() { const Elf_Ehdr &e = this->Obj.getHeader(); OS << "ELF Header:\n"; @@ -7340,18 +7306,3 @@ template <class ELFT> void LLVMELFDumper<ELFT>::printMipsABIFlags() { W.printFlags("Flags 1", Flags->flags1, makeArrayRef(ElfMipsFlags1)); W.printHex("Flags 2", Flags->flags2); } - -template <class ELFT> -void JSONELFDumper<ELFT>::printFileSummary(StringRef FileStr, ObjectFile &Obj, - ArrayRef<std::string> InputFilenames, - const Archive *A) { - FileScope = std::make_unique<DictScope>(this->W, FileStr); - DictScope D(this->W, "FileSummary"); - this->W.printString("File", FileStr); - this->W.printString("Format", Obj.getFileFormatName()); - this->W.printString("Arch", Triple::getArchTypeName(Obj.getArch())); - this->W.printString( - "AddressSize", - std::string(formatv("{0}bit", 8 * Obj.getBytesInAddress()))); - this->printLoadName(); -} diff --git a/llvm/tools/llvm-readobj/ObjDumper.cpp b/llvm/tools/llvm-readobj/ObjDumper.cpp index 6dde372..dc4a303 100644 --- a/llvm/tools/llvm-readobj/ObjDumper.cpp +++ b/llvm/tools/llvm-readobj/ObjDumper.cpp @@ -13,7 +13,6 @@ #include "ObjDumper.h" #include "llvm-readobj.h" -#include "llvm/Object/Archive.h" #include "llvm/Object/ObjectFile.h" #include "llvm/Support/Error.h" #include "llvm/Support/FormatVariadic.h" @@ -86,18 +85,6 @@ void ObjDumper::printAsStringList(StringRef StringContent, } } -void ObjDumper::printFileSummary(StringRef FileStr, object::ObjectFile &Obj, - ArrayRef<std::string> InputFilenames, - const object::Archive *A) { - W.startLine() << "\n"; - W.printString("File", FileStr); - W.printString("Format", Obj.getFileFormatName()); - W.printString("Arch", Triple::getArchTypeName(Obj.getArch())); - W.printString("AddressSize", - std::string(formatv("{0}bit", 8 * Obj.getBytesInAddress()))); - this->printLoadName(); -} - static std::vector<object::SectionRef> getSectionRefsByNameOrIndex(const object::ObjectFile &Obj, ArrayRef<std::string> Sections) { diff --git a/llvm/tools/llvm-readobj/ObjDumper.h b/llvm/tools/llvm-readobj/ObjDumper.h index a09a243..b395a95 100644 --- a/llvm/tools/llvm-readobj/ObjDumper.h +++ b/llvm/tools/llvm-readobj/ObjDumper.h @@ -20,7 +20,6 @@ namespace llvm { namespace object { -class Archive; class COFFImportFile; class ObjectFile; class XCOFFObjectFile; @@ -40,9 +39,6 @@ public: virtual bool canDumpContent() { return true; } - virtual void printFileSummary(StringRef FileStr, object::ObjectFile &Obj, - ArrayRef<std::string> InputFilenames, - const object::Archive *A); virtual void printFileHeaders() = 0; virtual void printSectionHeaders() = 0; virtual void printRelocations() = 0; diff --git a/llvm/tools/llvm-readobj/Opts.td b/llvm/tools/llvm-readobj/Opts.td index d0f273fa..7723691 100644 --- a/llvm/tools/llvm-readobj/Opts.td +++ b/llvm/tools/llvm-readobj/Opts.td @@ -28,7 +28,6 @@ def expand_relocs : FF<"expand-relocs", "Expand each shown relocation to multipl def file_header : FF<"file-header", "Display file header">; def headers : FF<"headers", "Equivalent to setting: --file-header, --program-headers, --section-headers">; defm hex_dump : Eq<"hex-dump", "Display the specified section(s) as hexadecimal bytes">, MetaVarName<"<name or index>">; -def pretty_print : FF<"pretty-print", "Pretty print JSON output">; def relocs : FF<"relocs", "Display the relocation entries in the file">; def section_data : FF<"section-data", "Display section data for each section shown. This option has no effect for GNU style output">; def section_details : FF<"section-details", "Display the section details">; @@ -48,7 +47,7 @@ def unwind : FF<"unwind", "Display unwind information">; def grp_elf : OptionGroup<"kind">, HelpText<"OPTIONS (ELF specific)">; def dynamic_table : FF<"dynamic-table", "Display the dynamic section table">, Group<grp_elf>; def elf_linker_options : FF<"elf-linker-options", "Display the .linker-options section">, Group<grp_elf>; -defm elf_output_style : Eq<"elf-output-style", "Specify ELF dump style: LLVM, GNU, JSON">, Group<grp_elf>; +defm elf_output_style : Eq<"elf-output-style", "Specify ELF dump style: LLVM or GNU">, Group<grp_elf>; def histogram : FF<"histogram", "Display bucket list histogram for hash sections">, Group<grp_elf>; def section_groups : FF<"section-groups", "Display section groups">, Group<grp_elf>; def gnu_hash_table : FF<"gnu-hash-table", "Display the GNU hash table for dynamic symbols">, Group<grp_elf>; diff --git a/llvm/tools/llvm-readobj/llvm-readobj.cpp b/llvm/tools/llvm-readobj/llvm-readobj.cpp index 46862bb..a598e2c 100644 --- a/llvm/tools/llvm-readobj/llvm-readobj.cpp +++ b/llvm/tools/llvm-readobj/llvm-readobj.cpp @@ -99,7 +99,6 @@ static bool DynamicSymbols; static bool FileHeaders; static bool Headers; static std::vector<std::string> HexDump; -static bool PrettyPrint; static bool PrintStackMap; static bool PrintStackSizes; static bool Relocations; @@ -231,17 +230,13 @@ static void parseOptions(const opt::InputArgList &Args) { opts::DynamicTable = Args.hasArg(OPT_dynamic_table); opts::ELFLinkerOptions = Args.hasArg(OPT_elf_linker_options); if (Arg *A = Args.getLastArg(OPT_elf_output_style_EQ)) { - std::string OutputStyleChoice = A->getValue(); - opts::Output = StringSwitch<opts::OutputStyleTy>(OutputStyleChoice) - .Case("LLVM", opts::OutputStyleTy::LLVM) - .Case("GNU", opts::OutputStyleTy::GNU) - .Case("JSON", opts::OutputStyleTy::JSON) - .Default(opts::OutputStyleTy::UNKNOWN); - if (opts::Output == opts::OutputStyleTy::UNKNOWN) { - error("--elf-output-style value should be either 'LLVM', 'GNU', or " - "'JSON', but was '" + - OutputStyleChoice + "'"); - } + StringRef V(A->getValue()); + if (V == "LLVM") + opts::Output = opts::OutputStyleTy::LLVM; + else if (V == "GNU") + opts::Output = opts::OutputStyleTy::GNU; + else + error("--elf-output-style value should be either 'LLVM' or 'GNU'"); } opts::GnuHashTable = Args.hasArg(OPT_gnu_hash_table); opts::HashSymbols = Args.hasArg(OPT_hash_symbols); @@ -249,7 +244,6 @@ static void parseOptions(const opt::InputArgList &Args) { opts::HashHistogram = Args.hasArg(OPT_histogram); opts::NeededLibraries = Args.hasArg(OPT_needed_libs); opts::Notes = Args.hasArg(OPT_notes); - opts::PrettyPrint = Args.hasArg(OPT_pretty_print); opts::ProgramHeaders = Args.hasArg(OPT_program_headers); opts::RawRelr = Args.hasArg(OPT_raw_relr); opts::SectionGroups = Args.hasArg(OPT_section_groups); @@ -339,7 +333,18 @@ static void dumpObject(ObjectFile &Obj, ScopedPrinter &Writer, reportError(DumperOrErr.takeError(), FileStr); Dumper = (*DumperOrErr).get(); - Dumper->printFileSummary(FileStr, Obj, opts::InputFilenames, A); + if (opts::Output == opts::LLVM || opts::InputFilenames.size() > 1 || A) { + Writer.startLine() << "\n"; + Writer.printString("File", FileStr); + } + if (opts::Output == opts::LLVM) { + Writer.printString("Format", Obj.getFileFormatName()); + Writer.printString("Arch", Triple::getArchTypeName(Obj.getArch())); + Writer.printString( + "AddressSize", + std::string(formatv("{0}bit", 8 * Obj.getBytesInAddress()))); + Dumper->printLoadName(); + } if (opts::FileHeaders) Dumper->printFileHeaders(); @@ -545,13 +550,6 @@ static void dumpInput(StringRef File, ScopedPrinter &Writer) { OwningBinary<Binary>(std::move(Bin), std::move(Buffer))); } -std::unique_ptr<ScopedPrinter> createWriter() { - if (opts::Output == opts::JSON) - return std::make_unique<JSONScopedPrinter>( - fouts(), opts::PrettyPrint ? 2 : 0, std::make_unique<ListScope>()); - return std::make_unique<ScopedPrinter>(fouts()); -} - int main(int argc, char *argv[]) { InitLLVM X(argc, argv); BumpPtrAllocator A; @@ -612,17 +610,16 @@ int main(int argc, char *argv[]) { opts::SectionHeaders = true; } - std::unique_ptr<ScopedPrinter> Writer = createWriter(); - + ScopedPrinter Writer(fouts()); for (const std::string &I : opts::InputFilenames) - dumpInput(I, *Writer.get()); + dumpInput(I, Writer); if (opts::CodeViewMergedTypes) { if (opts::CodeViewEnableGHash) - dumpCodeViewMergedTypes(*Writer.get(), CVTypes.GlobalIDTable.records(), + dumpCodeViewMergedTypes(Writer, CVTypes.GlobalIDTable.records(), CVTypes.GlobalTypeTable.records()); else - dumpCodeViewMergedTypes(*Writer.get(), CVTypes.IDTable.records(), + dumpCodeViewMergedTypes(Writer, CVTypes.IDTable.records(), CVTypes.TypeTable.records()); } diff --git a/llvm/tools/llvm-readobj/llvm-readobj.h b/llvm/tools/llvm-readobj/llvm-readobj.h index 0ea695d..7672da5 100644 --- a/llvm/tools/llvm-readobj/llvm-readobj.h +++ b/llvm/tools/llvm-readobj/llvm-readobj.h @@ -39,7 +39,7 @@ extern bool ExpandRelocs; extern bool RawRelr; extern bool CodeViewSubsectionBytes; extern bool Demangle; -enum OutputStyleTy { LLVM, GNU, JSON, UNKNOWN }; +enum OutputStyleTy { LLVM, GNU }; extern OutputStyleTy Output; } // namespace opts |