[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