[llvm] [llvm-objdump] Add inlined function display support (PR #142246)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 5 01:31:05 PDT 2025


================
@@ -150,44 +231,51 @@ void LiveVariablePrinter::addCompileUnit(DWARFDie D) {
 void LiveVariablePrinter::update(object::SectionedAddress ThisAddr,
                                  object::SectionedAddress NextAddr,
                                  bool IncludeDefinedVars) {
+  // Do not create live ranges when debug-inlined-funcs option is provided with
+  // line format option.
+  if (DbgInlinedFunctions == DVLine)
+    return;
+
   // First, check variables which have already been assigned a column, so
   // that we don't change their order.
-  SmallSet<unsigned, 8> CheckedVarIdxs;
+  SmallSet<unsigned, 8> CheckedElementIdxs;
   for (unsigned ColIdx = 0, End = ActiveCols.size(); ColIdx < End; ++ColIdx) {
-    if (!ActiveCols[ColIdx].isActive())
+    if (!ActiveCols[ColIdx].isActive()) {
       continue;
-    CheckedVarIdxs.insert(ActiveCols[ColIdx].VarIdx);
-    LiveVariable &LV = LiveVariables[ActiveCols[ColIdx].VarIdx];
-    ActiveCols[ColIdx].LiveIn = LV.liveAtAddress(ThisAddr);
-    ActiveCols[ColIdx].LiveOut = LV.liveAtAddress(NextAddr);
+    }
+    CheckedElementIdxs.insert(ActiveCols[ColIdx].ElementIdx);
+    auto &LE = LiveElements[ActiveCols[ColIdx].ElementIdx];
----------------
jh7370 wrote:

Again, please don't use `auto` here.

https://github.com/llvm/llvm-project/pull/142246


More information about the llvm-commits mailing list