[PATCH] D102265: [lld][WebAssembly] Remove relocation target verification

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 11 12:03:58 PDT 2021


sbc100 added inline comments.


================
Comment at: lld/test/wasm/reloc-addend.s:39
 # CHECK-NEXT:         Addend:          64
+# CHECK-NEXT:       - Type:            R_WASM_MEMORY_ADDR_I32
+# CHECK-NEXT:         Index:           0
----------------
dschuff wrote:
> Does it make sense to test negative addends for other reloc types too?
Yes.... although I might push that to a followup since this change is mostly about removing the checker.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102265



More information about the llvm-commits mailing list