diff options
Diffstat (limited to 'src/wasm/wasm-s-parser.cpp')
-rw-r--r-- | src/wasm/wasm-s-parser.cpp | 35 |
1 files changed, 10 insertions, 25 deletions
diff --git a/src/wasm/wasm-s-parser.cpp b/src/wasm/wasm-s-parser.cpp index a9fd2de77..ab6230442 100644 --- a/src/wasm/wasm-s-parser.cpp +++ b/src/wasm/wasm-s-parser.cpp @@ -2744,35 +2744,20 @@ Expression* SExpressionWasmBuilder::makeTupleExtract(Element& s) { } Expression* SExpressionWasmBuilder::makeCallRef(Element& s, bool isReturn) { - Index operandsStart = 1; - std::optional<HeapType> sigType; - try { - sigType = parseHeapType(*s[1]); - operandsStart = 2; - } catch (ParseException& p) { - // The type annotation is required for return_call_ref but temporarily - // optional for call_ref. - if (isReturn) { - throw; - } - } + HeapType sigType = parseHeapType(*s[1]); std::vector<Expression*> operands; - parseOperands(s, operandsStart, s.size() - 1, operands); + parseOperands(s, 2, s.size() - 1, operands); auto* target = parseExpression(s[s.size() - 1]); - if (sigType) { - if (!sigType->isSignature()) { - throw ParseException( - std::string(isReturn ? "return_call_ref" : "call_ref") + - " type annotation should be a signature", - s.line, - s.col); - } - return Builder(wasm).makeCallRef( - target, operands, sigType->getSignature().results, isReturn); + if (!sigType.isSignature()) { + throw ParseException( + std::string(isReturn ? "return_call_ref" : "call_ref") + + " type annotation should be a signature", + s.line, + s.col); } - return ValidatingBuilder(wasm, s.line, s.col) - .validateAndMakeCallRef(target, operands, isReturn); + return Builder(wasm).makeCallRef( + target, operands, sigType.getSignature().results, isReturn); } Expression* SExpressionWasmBuilder::makeI31New(Element& s) { |