[PATCH] D156742: [BOLT][DWARF] Fix handling inlined subroutine with no output pc.

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 31 14:33:54 PDT 2023


ayermolo added inline comments.


================
Comment at: bolt/lib/Rewrite/DWARFRewriter.cpp:842
     case dwarf::DW_TAG_catch_block: {
-      uint64_t RangesSectionOffset = RangesSectionWriter.getEmptyRangesOffset();
+      uint64_t RangesSectionOffset = 0;
       Expected<DWARFAddressRangesVector> RangesOrError =
----------------
maksfb wrote:
> Why is this change needed?
It's noop. If range is empty we will normalize to low_pc/high_pc.
In subsequent diffs I want to remove it completely.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156742



More information about the llvm-commits mailing list