[PATCH] llvm-readobj: COFF/ELF-specific low-level output

Eric Christopher echristo at gmail.com
Tue Mar 19 17:36:20 PDT 2013


  Looks pretty good so far. It could use a few example (if minimal) testcases to show that it's actually getting the correct things out of the object file. If you want you can create them from .s or .ll files or just check in the object files. I'd prefer the former because those are easier to look at and change/add to, but it's up to you.

  The code produces a ton of warnings and so doesn't compile for me using -Werror so I'm unable to take a look at the output much more than your unformatted example. If you could fix those first and repost I'd appreciate it.

  Thanks!

http://llvm-reviews.chandlerc.com/D416



More information about the llvm-commits mailing list