[PATCH] D154655: [DebugInfo] Force users of DWARFDebugAbbrev to call parse before iterating
    Alexander Yermolovich via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Jul 12 10:38:00 PDT 2023
    
    
  
ayermolo added inline comments.
================
Comment at: bolt/lib/Core/DebugData.cpp:1301
       if (AbbrevSetOffsets.empty()) {
+        const DWARFDebugAbbrev *DebugAbbrev =
+            Unit.getContext().getDebugAbbrev();
----------------
bulbazord wrote:
> ayermolo wrote:
> > Assuming my changes won't be reverted BOLT no longer uses this method.
> Good to know. For now, it was causing test failures so I'm going to land this with this change. It'll just go away when we're able to delete this method.
This BOLT code is no longer in trunk...
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154655/new/
https://reviews.llvm.org/D154655
    
    
More information about the llvm-commits
mailing list