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

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/include/llvm/DebugInfo/DWARF/DWARFContext.h:111-113
+    MACINFO = 1,
+    MACINFODWO,
+    MACRO
----------------
Why did you change the naming style of this? LLVM coding standards say that enums should use the same naming scheme as types: https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly.

Also, why the `= 1`?


================
Comment at: llvm/include/llvm/DebugInfo/DWARF/DWARFContext.h:114-115
+    MACRO
+    /* FIXME: Add support for
+   .debug_macro.dwo section */
+  };
----------------
Use C++ style comments, and put this all on one line. Also add missing trailing full stop.


================
Comment at: llvm/lib/DebugInfo/DWARF/DWARFContext.cpp:300
+  auto Macro = std::make_unique<DWARFDebugMacro>();
+  auto parseAndDump = [&](DWARFDataExtractor &Data, bool IsMacro) {
+    if (Error Err = Macro->parse(getStringExtractor(), Data, IsMacro))
----------------
`ParseAndDump`. This is a (callable) object, not a function.


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

https://reviews.llvm.org/D73086