[Lldb-commits] [lldb] [llvm] [lldb] Use llvm::DWARFExpression::iterator in DWARFExpression::Evaluate (PR #190556)

[Sergei Barannikov via lldb-commits]

================
@@ -2010,24 +2006,25 @@ llvm::Expected<Value> DWARFExpression::Evaluate(
     case DW_OP_implicit_value: {
       dwarf4_location_description_kind = Implicit;
 
-      const uint32_t len = opcodes.GetULEB128(&offset);
-      const void *data = opcodes.GetData(&offset, len);
+      const uint64_t block_size = op->getRawOperand(0);
+      uint64_t block_offset = op->getRawOperand(1);
----------------
s-barannikov wrote:

I checked with the standard, and yes, there is a little inconsistency with how LLVM represents variable-length operands. I'll add a comment.


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