diff options
| author | Chris Bieneman <beanz@apple.com> | 2016-06-27 19:53:53 +0000 |
|---|---|---|
| committer | Chris Bieneman <beanz@apple.com> | 2016-06-27 19:53:53 +0000 |
| commit | 8ff0c113575167809365d2de5f6a204ff9c9c2bb (patch) | |
| tree | bcfa2a036127dcd47a729941ec0389ec8022af0c /llvm/lib/ObjectYAML/ELFYAML.cpp | |
| parent | f0fa61ffcfc8550f1b4282e456b3763d71b60c76 (diff) | |
| download | llvm-8ff0c113575167809365d2de5f6a204ff9c9c2bb.zip llvm-8ff0c113575167809365d2de5f6a204ff9c9c2bb.tar.gz llvm-8ff0c113575167809365d2de5f6a204ff9c9c2bb.tar.bz2 | |
[yaml2obj] Remove --format option in favor of YAML tags
Summary:
Our YAML library's handling of tags isn't perfect, but it is good enough to get rid of the need for the --format argument to yaml2obj. This patch does exactly that.
Instead of requiring --format, it infers the format based on the tags found in the object file. The supported tags are:
!ELF
!COFF
!mach-o
!fat-mach-o
I have a corresponding patch that is quite large that fixes up all the in-tree test cases.
Reviewers: rafael, Bigcheese, compnerd, silvas
Subscribers: compnerd, llvm-commits
Differential Revision: http://reviews.llvm.org/D21711
llvm-svn: 273915
Diffstat (limited to 'llvm/lib/ObjectYAML/ELFYAML.cpp')
| -rw-r--r-- | llvm/lib/ObjectYAML/ELFYAML.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/ObjectYAML/ELFYAML.cpp b/llvm/lib/ObjectYAML/ELFYAML.cpp index a480783..521e237 100644 --- a/llvm/lib/ObjectYAML/ELFYAML.cpp +++ b/llvm/lib/ObjectYAML/ELFYAML.cpp @@ -820,6 +820,7 @@ void MappingTraits<ELFYAML::Relocation>::mapping(IO &IO, void MappingTraits<ELFYAML::Object>::mapping(IO &IO, ELFYAML::Object &Object) { assert(!IO.getContext() && "The IO context is initialized already"); IO.setContext(&Object); + IO.mapTag("!ELF", true); IO.mapRequired("FileHeader", Object.Header); IO.mapOptional("Sections", Object.Sections); IO.mapOptional("Symbols", Object.Symbols); |