[Lldb-commits] [lldb] [llvm] [NFC] Separate high-level-dependent portions of DWARFExpression (revised) (PR #143170)

[David Blaikie via lldb-commits]

================
@@ -425,7 +425,7 @@ void CFIProgram::printOperand(raw_ostream &OS, DIDumpOptions DumpOpts,
   case OT_Expression:
     assert(Instr.Expression && "missing DWARFExpression object");
     OS << " ";
-    Instr.Expression->print(OS, DumpOpts, nullptr);
+    DWARFExpressionPrinter::print(&(*Instr.Expression), OS, DumpOpts, nullptr);
----------------
dwblaikie wrote:

Ahh, right, std::optional when I was thinking of std::unique_ptr's API. Sorry about that. FWIW in that case I don't mind either way.

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