[PATCH] D77353: [WebAssembly] Add DW_OP_WASM_location_int

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 16 13:56:26 PDT 2020


aardappel added a comment.

Ok, will try to make it more consistently unsigned, given that down the line they will be stored as unsigned anyway. But its still messy, given that at least thru one path they arrive as this `TargetIndexLocation` which is signed.
Made it explicit in the comments for `TargetIndex` that these are meant to be unsigned.


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

https://reviews.llvm.org/D77353





More information about the llvm-commits mailing list