[PATCH] D52813: [WebAssembly] Saturating arithmetic intrinsics

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 3 14:42:28 PDT 2018


aheejin added inline comments.


================
Comment at: test/CodeGen/WebAssembly/simd-arith.ll:6
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-disable-explicit-locals -wasm-keep-registers -mattr=-simd128 | FileCheck %s --check-prefixes CHECK,NO-SIMD128,NO-SIMD128-SLOW
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-disable-explicit-locals -wasm-keep-registers -mattr=-simd128 -fast-isel | FileCheck %s --check-prefixes CHECK,NO-SIMD128,NO-SIMD128-FAST
 
----------------
Do we need both the slow and fast versions for `NO-SIMD128` too? Aren't they the same anyway?


Repository:
  rL LLVM

https://reviews.llvm.org/D52813





More information about the llvm-commits mailing list