[Lldb-commits] [lldb] [lldb] Skip local variable declarations at start of Wasm function (PR #190093)

[Jonas Devlieghere via lldb-commits]

================
@@ -365,13 +389,20 @@ ParseFunctions(DataExtractor &data) {
   functions.reserve(*function_count);
 
   for (uint32_t i = 0; i < *function_count; ++i) {
-    llvm::Expected<uint32_t> function_size = GetULEB32(data, offset);
-    if (!function_size)
-      return function_size.takeError();
     // llvm-objdump considers the ULEB with the function size to be part of the
     // function. We can't do that here because that would break symbolic
     // breakpoints, as that address is never executed.
----------------
JDevlieghere wrote:

Updated to say that it wouldn't match the DWARF.

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