Add Int32x4 Wasm Simd Binops, compare ops, select

src/ia32/assembler-ia32-inl.h

Index: src/ia32/assembler-ia32-inl.h
diff --git a/src/ia32/assembler-ia32-inl.h b/src/ia32/assembler-ia32-inl.h
index 5225229f202fc734ebe57207a1100747bcdbc10a..d8a107b113d1a1ab057e472e8ce6bb739fe5f87c 100644
--- a/src/ia32/assembler-ia32-inl.h
+++ b/src/ia32/assembler-ia32-inl.h
@@ -48,7 +48,7 @@ namespace internal {
 
 bool CpuFeatures::SupportsCrankshaft() { return true; }
 
-bool CpuFeatures::SupportsSimd128() { return true; }
+bool CpuFeatures::SupportsWasmSimd128() { return IsSupported(SSE4_1); }

[bbudge, 2017/03/14 22:19:29]

I think this could be weaker with the current code: IsSupported(SSE3).

cc'ing Jing to verify.

[gdeepti, 2017/03/14 22:26:09]

Defer to Jing here, but added this with the assumption that to support a
complete set of opcodes as in the current Wasm spec, SSE4.1 support will be
required. 

[Jing, 2017/03/15 08:20:33]

I think the strict check for SSE4_1 is needed, because some WASM SIMD opcodes
will use SSE4_1 instructions. Otherwise we have to emulate all of them with
SSE/SSE2 in order to support older platforms.

[gdeepti, 2017/03/15 13:23:03]

Thanks for confirming, will leave this as a check for SSE4_1

 
 static const byte kCallOpcode = 0xE8;
 static const int kNoCodeAgeSequenceLength = 5;