[MERGE #5839 @penzn] [WASM SIMD update, part 1] tests cleanup

Merge pull request #5839 from penzn:wasm.simd.tests

For #5838. All changes to WASM files are compatible with WebAssembly/wabt@048e01a.

Author: MikeHolman

lib/WasmReader/WasmBinaryOpcodesSimd.h

@@ -59,7 +59,7 @@ WASM_EXTRACTLANE_OPCODE(I8ExtractLaneS, __prefix | 0x0b, I_M128, Simd128_Extract
 WASM_EXTRACTLANE_OPCODE(I8ExtractLaneU, __prefix | 0x0c, I_M128, Simd128_ExtractLane_U8, true, "i16x8.extract_lane_u")
 WASM_EXTRACTLANE_OPCODE(I4ExtractLane, __prefix | 0x0d, I_M128, Simd128_ExtractLane_I4, true, "i32x4.extract_lane")
 WASM_EXTRACTLANE_OPCODE(I2ExtractLane, __prefix | 0x0e, L_M128, Simd128_ExtractLane_I2, true, "i64x2.extract_lane")
-WASM_EXTRACTLANE_OPCODE(F4ExtractLane, __prefix | 0x0f, I_M128, Simd128_ExtractLane_F4, true, "f32x4.extract_lane")
+WASM_EXTRACTLANE_OPCODE(F4ExtractLane, __prefix | 0x0f, F_M128, Simd128_ExtractLane_F4, true, "f32x4.extract_lane")
 WASM_BINARY_OPCODE(F2ExtractLane, __prefix | 0x10, D_M128_I, Simd128_ExtractLane_D2, true, "f64x2.extract_lane")
 WASM_REPLACELANE_OPCODE(I16ReplaceLane, __prefix | 0x11, M128_I, Simd128_ReplaceLane_I16, true, "i8x16.replace_lane")
 WASM_REPLACELANE_OPCODE(I8ReplaceLane, __prefix | 0x12, M128_I, Simd128_ReplaceLane_I8, true, "i16x8.replace_lane")

lib/WasmReader/WasmByteCodeGenerator.cpp

@@ -1571,6 +1571,7 @@ EmitInfo WasmBytecodeGenerator::EmitReplaceLaneExpr(Js::OpCodeAsmJs op, const Wa
 
     m_writer->AsmReg4(op, resultReg, simdArg.location, indexInfo.location, valueArg.location);
     ReleaseLocation(&indexInfo);
+    ReleaseLocation(&valueArg);
     return result;
 }
 
@@ -1579,6 +1580,11 @@ EmitInfo WasmBytecodeGenerator::EmitM128BitSelect()
     EmitInfo mask = PopEvalStack(WasmTypes::M128);
     EmitInfo arg2Info = PopEvalStack(WasmTypes::M128);
     EmitInfo arg1Info = PopEvalStack(WasmTypes::M128);
+
+    ReleaseLocation(&mask);
+    ReleaseLocation(&arg2Info);
+    ReleaseLocation(&arg1Info);
+
     Js::RegSlot resultReg = GetRegisterSpace(WasmTypes::M128)->AcquireTmpRegister();
     EmitInfo resultInfo(resultReg, WasmTypes::M128);
     m_writer->AsmReg4(Js::OpCodeAsmJs::Simd128_BitSelect_I4, resultReg, arg1Info.location, arg2Info.location, mask.location);
@@ -1590,6 +1596,8 @@ EmitInfo WasmBytecodeGenerator::EmitV8X16Shuffle()
     EmitInfo arg2Info = PopEvalStack(WasmTypes::M128);
     EmitInfo arg1Info = PopEvalStack(WasmTypes::M128);
 
+    // FIXME Release arg2Info and arg1Info
+
     Js::RegSlot resultReg = GetRegisterSpace(WasmTypes::M128)->AcquireTmpRegister();
     EmitInfo resultInfo(resultReg, WasmTypes::M128);
 
@@ -1612,16 +1620,16 @@ EmitInfo WasmBytecodeGenerator::EmitExtractLaneExpr(Js::OpCodeAsmJs op, const Wa
     WasmTypes::WasmType simdArgType = signature[1];
 
     EmitInfo simdArgInfo = PopEvalStack(simdArgType, _u("Argument should be of type M128"));
+    ReleaseLocation(&simdArgInfo);
 
     Js::RegSlot resultReg = GetRegisterSpace(resultType)->AcquireTmpRegister();
     EmitInfo resultInfo(resultReg, resultType);
 
     //put index into a register to reuse the existing infra in Interpreter and Compiler
     EmitInfo indexInfo = EmitLaneIndex(op);
+    ReleaseLocation(&indexInfo);
 
     m_writer->AsmReg3(op, resultReg, simdArgInfo.location, indexInfo.location);
-    ReleaseLocation(&indexInfo);
-    ReleaseLocation(&simdArgInfo);
     return resultInfo;
 }