[PATCH] D125181: [BOLT][DWARF] Fix assert for split dwarf.

[Alexander Yermolovich via Phabricator via llvm-commits]

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


================
Comment at: bolt/lib/Rewrite/DWARFRewriter.cpp:1021-1028
       } else if (CU->getVersion() >= 5) {
         // A case where we were not using .debug_addr section, but after update
         // now using it.
         const DWARFAbbreviationDeclaration *Abbrev =
             DIE.getAbbreviationDeclarationPtr();
         AbbrevWriter->addAttribute(*CU, Abbrev, dwarf::DW_AT_addr_base,
                                    dwarf::DW_FORM_sec_offset);
----------------
rafauler wrote:
> Won't this code be unreachable after the change in 1003?
ah yes, 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125181