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

[Derek Schuff via Phabricator via llvm-commits]

dschuff added a comment.



> Ok, and it would change `TI_LOCAL_INDIRECT` back to `TI_LOCAL` as it emits `DW_OP_deref` ?

Yes... or maybe not? i.e. does it actually matter after that what value the TargetIndex MachineOperand has?

> And `TI_LOCAL_INDIRECT` is now marked as being internal to LLVM only?

or maybe another way to say it is that the TargetIndex MachineOperand is itself an internal LLVM data structure, and it's only by happenstance that one of its members has values that can match up with DW_OP_WASM_location values?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94347