[PATCH] D85552: [WebAssembly] wasm64: fix memory.init operand types

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 13:26:45 PDT 2020


aardappel created this revision.
aardappel added reviewers: sbc100, dschuff.
Herald added subscribers: llvm-commits, sunfish, hiraditya, jgravelle-google.
Herald added a project: LLVM.
aardappel requested review of this revision.
Herald added a subscriber: aheejin.

I had assumed they would all become in i64, but this is not necessary as long as data segments stay 32-bit, see:
https://github.com/WebAssembly/memory64/blob/master/proposals/memory64/Overview.md


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85552

Files:
  lld/wasm/InputChunks.cpp
  lld/wasm/OutputSections.cpp
  lld/wasm/OutputSegment.h
  lld/wasm/SymbolTable.cpp
  lld/wasm/SymbolTable.h
  lld/wasm/Writer.cpp
  llvm/lib/Target/WebAssembly/WebAssemblyInstrBulkMemory.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85552.284012.patch
Type: text/x-patch
Size: 4717 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200807/f86e9f1b/attachment.bin>


More information about the llvm-commits mailing list