[PATCH] D94347: [WebAssembly] locals can now be indirect in DWARF

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 17:12:40 PST 2021


aardappel added a comment.

> the only difference would be that instead of encoding TI_LOCAL_INDIRECT directly into first argument of DW_OP_WASM_location, that the Dwarf writer would instead change the final argument of DW_AT_location to DW_OP_deref. That change is less elegant in the LLVM code but makes the standardized interface simpler.

Ok, and it would change `TI_LOCAL_INDIRECT` back to `TI_LOCAL` as it emits `DW_OP_deref` ? And `TI_LOCAL_INDIRECT` is now marked as being internal to LLVM only?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94347



More information about the llvm-commits mailing list