[PATCH] D22686: [WASM] SIMD128 support.

Tue Jul 26 09:35:16 PDT 2016

sunfish added inline comments.

================
Comment at: lib/Target/WebAssembly/WebAssemblySubtarget.cpp:69
@@ +68,3 @@
+    if (HasSIMD128)
+      return true;
+    return false;
----------------
It's not necessary to check HasSIMD128 here, because we shouldn't have created an ARGUMENT_v16i8 etc. if we don't have HasSIMD128 in the first place :-). And without that check, this function doesn't need to be a member of WebAssemblySubtarget.

The same applies to other hasSIMD128() checks in the patch as well.

Repository:
  rL LLVM

https://reviews.llvm.org/D22686