[PATCH] D87263: [DWARFYAML] Make the debug_ranges section optional.

[Xing GUO via Phabricator via llvm-commits]

Higuoxing added inline comments.


================
Comment at: llvm/test/ObjectYAML/MachO/DWARF-debug_ranges.yaml:288
+
+## Test generating and dumping a __debug_ranges section whose size isn't a multiple of address size.
+## This test case is to ensure that when the parser fails, the content of the __debug_ranges section
----------------
jhenderson wrote:
> This test case doesn't feel like it really belongs as part of this patch. It should be its own patch, since it isn't to do with writing an empty .debug_ranges section, right?
Yes, it doesn't belong to this patch. I added it because I touched the parser in dwarf2yaml.cpp. When the parser fails the content in the DWARF tag should remain empty. I will add it in a follow-up patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87263