[PATCH] D90503: [WebAssembly] Fixed DWARF DW_AT_low_pc encoded as 64-bit in wasm64

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 30 15:54:21 PDT 2020


dschuff added inline comments.


================
Comment at: llvm/test/MC/WebAssembly/dwarfdump64.ll:38
+; CHECK-NEXT:                DW_AT_decl_line	(5)
+; CHECK-NEXT:                DW_AT_location	(DW_OP_addr 0x8)
+
----------------
dschuff wrote:
> I have a feeling that this is probably wrong (i.e. it's 4 instead of 8). Maybe we should put a TODO here.
(I mean that it's using a 4-byte address instead of an 8-byte. not that the value is wrong)


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

https://reviews.llvm.org/D90503



More information about the llvm-commits mailing list