Commit 1e6fed5f authored by Andreas Haas's avatar Andreas Haas Committed by Commit Bot

[wasm] Remove obsolete CheckHasMemoryForAtomics

This function is equivalent to CheckHasMemory by now.

R=clemensb@chromium.org

Bug: v8:11074
Change-Id: I05fbd160d6e5b8d7433133052a8390521564632b
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2509590Reviewed-by: 's avatarClemens Backes <clemensb@chromium.org>
Commit-Queue: Andreas Haas <ahaas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#70896}
parent 874227b5
......@@ -2180,16 +2180,6 @@ class WasmFullDecoder : public WasmDecoder<validate> {
return true;
}
bool CheckHasMemoryForAtomics() {
if (CheckHasMemory()) return true;
if (!VALIDATE(this->module_->has_shared_memory)) {
this->DecodeError(this->pc_ - 1,
"Atomic opcodes used without shared memory");
return false;
}
return true;
}
bool CheckSimdPostMvp(WasmOpcode opcode) {
if (!FLAG_wasm_simd_post_mvp && WasmOpcodes::IsSimdPostMvpOpcode(opcode)) {
this->DecodeError(
......@@ -4061,7 +4051,7 @@ class WasmFullDecoder : public WasmDecoder<validate> {
this->DecodeError("invalid atomic opcode");
return 0;
}
if (!CheckHasMemoryForAtomics()) return 0;
if (!CheckHasMemory()) return 0;
MemoryAccessImmediate<validate> imm(
this, this->pc_ + opcode_length,
ElementSizeLog2Of(memtype.representation()));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment