Does obj2yaml support generating YAML from executables? Last time I checked it did not. This change only relates to executable.

Thanks for doing this.

The first thing I would say is that it really needs a clang-format pass as well as a lot more tests, since the change in functionality is not small.

This was clang-formatted, may be my clang format python script is old.

ping..

Made test case a yaml instead of binary.

Updated diff with latest tip, made few changes as recommended.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@278725 91177308-0d34-0410-b5e6-96231b3b80d8

Incorporated suggestions from previous diff.
Added ARM test.

Ping!

@compnerd. You are right, IR is better choice for test. Formatted the test too. Please +1 the change.

The test is now IR instead of asm. clang formatted the test.

@compnerd: Please review the changes. Took your suggestions into account.

Removed hexagon changes to leave only generic pretty printer changes. Hexagon will be added as a separate patch.

In D22932#510698, @compnerd wrote:

I think it would be better to split the hexagon support into a separate (trivial) patch. That would make this change only add the source and line number printing and tests for x86. The hexagon change would then add that target as well.

ping!

Made changes so all pretty printers can interleave source with disassembly. Added tests that test X86 and Hexagon printers.

Thanks. Will try that an send for review.

Thanks for the reminder :)

I am pushing this in as there is not a major design here. Just adding something that fell through the cracks when implementing other relocation printing method.

ping

Updated test.

test/tools/llvm-readobj/Inputs/phdrs-elf.exe-* files are checked in as Binary as they are fully formed executables. They also have sufficient mix of program headers which have TLS sections in permissible segments and some extra non PT_LOAD segments.

LGTM.

Incorporated the comments.

The patch to upstream the processor specific flags for small area is not yet in the Hexagon backend. That is why I made a binary. I do not know of other backends that can get me this flag.

Added test case.

Looks like I forgot to add the test case. Will update the diff.

ping.

I will make the blakent style change after this patch. For now I am making the changes you mentioned.

Addressed comments

Use AltName for section type string.