Commit 83544915 authored by Clemens Hammacher's avatar Clemens Hammacher Committed by Commit Bot

[wasm] Remove unused signature parameter for UnOp and BinOp

Neither the TF backend nor Liftoff use the signature, thus remove it.

R=titzer@chromium.org

Bug: v8:8423
Change-Id: I909e9a0095cac67aaefaebcb4240f7d9829e4c87
Reviewed-on: https://chromium-review.googlesource.com/c/1373777Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#58190}
parent 5f8cd45b
...@@ -632,8 +632,8 @@ class LiftoffCompiler { ...@@ -632,8 +632,8 @@ class LiftoffCompiler {
__ PushRegister(dst_type, dst); __ PushRegister(dst_type, dst);
} }
void UnOp(FullDecoder* decoder, WasmOpcode opcode, FunctionSig*, void UnOp(FullDecoder* decoder, WasmOpcode opcode, const Value& value,
const Value& value, Value* result) { Value* result) {
#define CASE_I32_UNOP(opcode, fn) \ #define CASE_I32_UNOP(opcode, fn) \
case WasmOpcode::kExpr##opcode: \ case WasmOpcode::kExpr##opcode: \
EmitUnOp<kWasmI32, kWasmI32>( \ EmitUnOp<kWasmI32, kWasmI32>( \
...@@ -786,8 +786,8 @@ class LiftoffCompiler { ...@@ -786,8 +786,8 @@ class LiftoffCompiler {
} }
} }
void BinOp(FullDecoder* decoder, WasmOpcode opcode, FunctionSig*, void BinOp(FullDecoder* decoder, WasmOpcode opcode, const Value& lhs,
const Value& lhs, const Value& rhs, Value* result) { const Value& rhs, Value* result) {
#define CASE_I32_BINOP(opcode, fn) \ #define CASE_I32_BINOP(opcode, fn) \
case WasmOpcode::kExpr##opcode: \ case WasmOpcode::kExpr##opcode: \
return EmitBinOp<kWasmI32, kWasmI32>( \ return EmitBinOp<kWasmI32, kWasmI32>( \
......
...@@ -681,9 +681,9 @@ struct ControlWithNamedConstructors : public ControlBase<Value> { ...@@ -681,9 +681,9 @@ struct ControlWithNamedConstructors : public ControlBase<Value> {
F(PopControl, Control* block) \ F(PopControl, Control* block) \
F(EndControl, Control* block) \ F(EndControl, Control* block) \
/* Instructions: */ \ /* Instructions: */ \
F(UnOp, WasmOpcode opcode, FunctionSig*, const Value& value, Value* result) \ F(UnOp, WasmOpcode opcode, const Value& value, Value* result) \
F(BinOp, WasmOpcode opcode, FunctionSig*, const Value& lhs, \ F(BinOp, WasmOpcode opcode, const Value& lhs, const Value& rhs, \
const Value& rhs, Value* result) \ Value* result) \
F(I32Const, Value* result, int32_t value) \ F(I32Const, Value* result, int32_t value) \
F(I64Const, Value* result, int64_t value) \ F(I64Const, Value* result, int64_t value) \
F(F32Const, Value* result, float value) \ F(F32Const, Value* result, float value) \
...@@ -2789,7 +2789,7 @@ class WasmFullDecoder : public WasmDecoder<validate> { ...@@ -2789,7 +2789,7 @@ class WasmFullDecoder : public WasmDecoder<validate> {
auto val = Pop(0, sig->GetParam(0)); auto val = Pop(0, sig->GetParam(0));
auto* ret = auto* ret =
sig->return_count() == 0 ? nullptr : Push(sig->GetReturn(0)); sig->return_count() == 0 ? nullptr : Push(sig->GetReturn(0));
CALL_INTERFACE_IF_REACHABLE(UnOp, opcode, sig, val, ret); CALL_INTERFACE_IF_REACHABLE(UnOp, opcode, val, ret);
break; break;
} }
case 2: { case 2: {
...@@ -2797,7 +2797,7 @@ class WasmFullDecoder : public WasmDecoder<validate> { ...@@ -2797,7 +2797,7 @@ class WasmFullDecoder : public WasmDecoder<validate> {
auto lval = Pop(0, sig->GetParam(0)); auto lval = Pop(0, sig->GetParam(0));
auto* ret = auto* ret =
sig->return_count() == 0 ? nullptr : Push(sig->GetReturn(0)); sig->return_count() == 0 ? nullptr : Push(sig->GetReturn(0));
CALL_INTERFACE_IF_REACHABLE(BinOp, opcode, sig, lval, rval, ret); CALL_INTERFACE_IF_REACHABLE(BinOp, opcode, lval, rval, ret);
break; break;
} }
default: default:
......
...@@ -206,13 +206,13 @@ class WasmGraphBuildingInterface { ...@@ -206,13 +206,13 @@ class WasmGraphBuildingInterface {
void EndControl(FullDecoder* decoder, Control* block) { ssa_env_->Kill(); } void EndControl(FullDecoder* decoder, Control* block) { ssa_env_->Kill(); }
void UnOp(FullDecoder* decoder, WasmOpcode opcode, FunctionSig* sig, void UnOp(FullDecoder* decoder, WasmOpcode opcode, const Value& value,
const Value& value, Value* result) { Value* result) {
result->node = BUILD(Unop, opcode, value.node, decoder->position()); result->node = BUILD(Unop, opcode, value.node, decoder->position());
} }
void BinOp(FullDecoder* decoder, WasmOpcode opcode, FunctionSig* sig, void BinOp(FullDecoder* decoder, WasmOpcode opcode, const Value& lhs,
const Value& lhs, const Value& rhs, Value* result) { const Value& rhs, Value* result) {
auto node = BUILD(Binop, opcode, lhs.node, rhs.node, decoder->position()); auto node = BUILD(Binop, opcode, lhs.node, rhs.node, decoder->position());
if (result) result->node = node; if (result) result->node = node;
} }
......
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