MrEven132 wrote: @echristo I revisited the helper structure and kept the cursor checks directly in the signed/unsigned helpers, removing `finishLEB128` and the extra wrapper lambdas. I also consolidated operand assignment and failure handling and reused the existing LLDB test helper.
This makes the code at the spot you pointed out a little more verbose, but I think removing those extra helper layers makes the patch simpler overall. Does this version look reasonable to you? https://github.com/llvm/llvm-project/pull/219149 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
