[PATCH] D52450: [Intrinsic] Add llvm.minnan and llvm.maxnan instrinsic functions

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 27 14:55:01 PDT 2018


tlively planned changes to this revision.
tlively marked an inline comment as done.
tlively added a comment.

WebAssembly's handling of +0/-0 *does* match IEEE 754-2018, so these intrinsics (whether target-independent or not) need to be changed to match those semantics as well. I will have to investigate whether the fminnan and fmaxnan ISel DAG nodes are already being used in ways that do not respect these semantics. If they are, we will need new ISel DAG nodes as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D52450





More information about the llvm-commits mailing list