[PATCH] D73086: [DWARF5] Added support for debug_macro section parsing and dumping in llvm-dwarfdump.

Sourabh Singh Tomar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 7 05:07:06 PST 2020


SouraVX added a comment.

@Ikudrin, now distinguishing macro/macinfo while parsing/dumping is not dependent on `Context`, that suggests we can reduce the test to `debug-macro-v5.s` bare minimum just `macro` section and `str` section. Reducing now makes more sense also since coverage of  `DW_AT_macros` attribute/value is now done in new test case `dwarfdump-macro-macinfo.test`.

Should we consider adding "Unsupported" test cases as asm or raw objects just like 'dwarfdump-macro-macinfo.test' ?? Please share your opinion on this!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73086/new/

https://reviews.llvm.org/D73086





More information about the llvm-commits mailing list