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

[Greg Clayton via Phabricator via lldb-commits]

clayborg added a comment.

Could we just always use memory reading and have the address contain more info? Right now you have the top 32 bits for the module ID. Could it be something like:

  struct WasmAddress {
    uint64_t module_id:16;
    uint64_t space:4; // 0 == code, 1 == data, 2 == global, 3==local, 4 == stack
    uint64_t frame_id:??;
    uint64_t addr: ??;
  }

This would be a bitfield that would all fit into a 64 bit value and could then be easily sent to the GDB server with the standard m and M packets.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78801