[PATCH] D82073: [ObjectYAML][ELF] Add support for emitting the .debug_info section.

Thu Jun 18 03:46:36 PDT 2020

Higuoxing marked an inline comment as done.
Higuoxing added inline comments.

================
Comment at: llvm/test/tools/yaml2obj/ELF/DWARF/debug-info.yaml:456
+
+## g) Test that all the properties can be overridden by the section header when
+## the "debug_info" entry exists.
----------------
jhenderson wrote:
> Higuoxing wrote:
> > In order not to bring too much difficulty. The common tests are very similar to the tests in other DWARF sections, except this one. The property of the `.debug_abbrev` section is overridden as well in this test.
> I'm not sure I follow. What's the point of overriding .debug_abbrev too? It will at least need explaining in a comment.
I want to test that `yaml2obj` is still able to overwrite the section header fields of the `.debug_abbrev` section when the `.debug_info` section exists. Does it make sense here?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82073