[PATCH] D52385: [WebAssembly] Add llvm.wasm.{min|max} intrinsic functions

[Thomas Lively via Phabricator via llvm-commits]

tlively added a comment.

The big question I have about this CL is whether these intrinsics should be target-independent instead of specific to the wasm. Their functionality is certainly not wasm-specific. @dschuff what do you think? I would just have to rename them and add documentation to the language reference. The downside is that I expect getting this merged as target independent intrinsics would be more difficult.


Repository:
  rL LLVM

https://reviews.llvm.org/D52385