[PATCH] D156958: [BOLT][DWARF] Fix handling CU with DW_AT_ranges

Maksim Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 01:47:03 PDT 2023


maksfb accepted this revision.
maksfb added a comment.
This revision is now accepted and ready to land.

LGTM. Please revise the summary before committing.



================
Comment at: bolt/lib/Rewrite/DWARFRewriter.cpp:1249
+      std::optional<uint64_t> DWOId = Unit.getDWOId();
+      assert(DWOId && "DWOId is not found for DWO Unit.");
       CurRangeBase = getDwoRangesBase(*DWOId);
----------------
Are you checking DWOId somewhere else? If not, convert to error. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156958



More information about the llvm-commits mailing list