[PATCH] D52372: [WebAssembly] SIMD conversions

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added a comment.

1. Is it correct to translate those llvm instructions to saturating versions of wasm instructions? For example, for scalars, we had a patch <https://github.com/llvm-mirror/llvm/commit/9f86840c1ccefbede8c0deeac9bb4f1a28608fc4> that implemented the saturating behavior. Don't we need a similar thing for vectors too?
2. Is this behavior also dependent on `nontrapping-fptoint` <https://github.com/llvm-mirror/llvm/blob/59895038204bec7cb4f924acf25b91f3056ea492/lib/Target/WebAssembly/WebAssembly.td#L30-L33> feature, as the scalar instructions <https://github.com/llvm-mirror/llvm/blob/59895038204bec7cb4f924acf25b91f3056ea492/lib/Target/WebAssembly/WebAssemblyInstrConv.td#L63-L104>?


Repository:
  rL LLVM

https://reviews.llvm.org/D52372