[PATCH] D74923: [DebugInfo]: Do not start parsing macinfo/macinfo.dwo if the section's are empty.

[Sourabh Singh Tomar via Phabricator via llvm-commits]

SouraVX added a comment.

> OK, so this is a subtle divergence between how debug_macinfo is parsed and debug_macro is parsed, even though they share some code. With macinfo the looping happens here in this function and with macro the looping happens in the caller to this in dumpMacroSection.



> Please fix this so macro and macinfo are handled in similar/the same way - both should build a single DWARFDebugMacro object representing all the contributions, then dump them together from there. (look at the way the existing macinfo support is written in DWARFContext - starting there/making the macro support similar)

That's your remark on D73086 <https://reviews.llvm.org/D73086>, Please correct me if I've mis-understood the context here-
here `if (Macro)` -- Macro is `std::unique_ptr<DWARFDebugMacro> Macro` for dumping, as of now it's used for macinfo/dwo parsing/dumping. You mentioned `both should build a single DWARFDebugMacro object representing all the contributions` so that's what I'm trying to achieve here, macinfo is not present then try constructing macro if present.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74923