[PATCH] D44393: [WebAssembly] Add missing implementation for --initial/max-memory args

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: lld/trunk/wasm/Writer.cpp:611
+  if (Config->InitialMemory != 0) {
+    if (Config->InitialMemory != alignTo(Config->InitialMemory, WasmPageSize))
+      error("initial memory must be " + Twine(WasmPageSize) + "-byte aligned");
----------------
This can be `Config->InitMemory % WasmPageSize != 0`


================
Comment at: lld/trunk/wasm/Writer.cpp:612
+    if (Config->InitialMemory != alignTo(Config->InitialMemory, WasmPageSize))
+      error("initial memory must be " + Twine(WasmPageSize) + "-byte aligned");
+    if (MemoryPtr > Config->InitialMemory)
----------------
Something like

  error: -initial-memory=<value>: value is not aligned to 4096 bytes

is more common style of an error message.


================
Comment at: lld/trunk/wasm/Writer.cpp:615
+      error("initial memory too small, " + Twine(MemoryPtr) + " bytes needed");
+    else
+      MemoryPtr = Config->InitialMemory;
----------------
This `else` looks odd because if there is an error in the first `if`, the control reaches here, but if there is an error in the second `if`, it doesn't. Maybe we should just set it unconditionally. We won't use a value if there's an error because we'll exit before we use a dummy value.


Repository:
  rL LLVM

https://reviews.llvm.org/D44393