[PATCH] D77353: [WebAssembly] Add DW_OP_WASM_location_int

Yury Delendik via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 9 12:15:36 PDT 2020


yurydelendik added a comment.

> If there are no further comments on improvements we can carry out right now, does anyone feel like approving this for merging? :)

The adding of new OP code `DW_OP_WASM_location_int` seems driven by LLVM implementation detail. I feel that is not necessary. Is it possible to use the alternative encoding suggested above, since we solved plumbing symbol support?


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

https://reviews.llvm.org/D77353





More information about the llvm-commits mailing list