[Lldb-commits] [PATCH] D78801: [LLDB] Add class WasmProcess for WebAssembly debugging

Fri Jan 28 20:11:13 PST 2022

xujuntwt95329 added a comment.

In D78801#3279811 <https://reviews.llvm.org/D78801#3279811>, @paolosev wrote:

> Thanks @xujuntwt95329! I am very happy that this was useful for WebAssembly Micro Runtime!

I believe we are making the world of WebAssembly better!

BTW, I find an issue when trying to debug multi-thread wasm app:
The `qWasmLocal` package doesn't contain the thread id, which means it can only get locals of the current thread. If we have thread A, B and C, and they stopped at the same time, then LLDB will send three `qWasmLocal` package, and the wasm runtime will give same reply.

I think we should add thread id into `qWasmLocal` so that wasm runtime will know which thread to process, Am I right?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78801/new/

https://reviews.llvm.org/D78801