[PATCH] D77353: [WebAssembly] Add DW_OP_WASM_location_int

Mon Apr 13 22:56:37 PDT 2020

paolosev added a comment.

In D77353#1978576 <https://reviews.llvm.org/D77353#1978576>, @aardappel wrote:

> @paolosev you may have an opinion on this, given that you're already working with this data as well?

I also think that if possible we should try to use only one code, since there are only 10 vendor extension codes left (9 if we count our `DW_OP_WASM_location`).

I am not an expert here, but looking at `DW_OP_implicit_value` as Yury says, it should be possible to have variable-length arguments (DW_OP_implicit_value expects a ULEB128 size followed by block of that size).

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

https://reviews.llvm.org/D77353