[PATCH] D130812: [BOLT][DWARF] Fix handling of multiple DW_OP_addrx in an expression

Maksim Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 29 18:38:03 PDT 2022


maksfb added inline comments.


================
Comment at: bolt/lib/Rewrite/DWARFRewriter.cpp:669
             }
+            // Updating the size of the experssion.
+            if (Updated)
----------------



================
Comment at: bolt/lib/Rewrite/DWARFRewriter.cpp:671
+            if (Updated)
+              DebugInfoPatcher.addUDataPatch(AttrOffset, SizeDiff + CurrEndOffset, 1);
           }
----------------
Do you have to update when `SizeDiff == 0`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130812



More information about the llvm-commits mailing list