[PATCH] D71218: [FPEnv] Remove rounding mode argument for constrained minnum/maxnum

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 02:59:44 PST 2019


uweigand added a comment.

In D71218#1775689 <https://reviews.llvm.org/D71218#1775689>, @kpn wrote:

> Random thought: Do we need to document which version of 754 this conforms to?


This seems to be a much more complicated topic than I expected, there are different versions of IEEE 754 and different versions of the C standard and different implementations in various libc versions, and none of them completely agree with each other ...   In any case, to avoid duplication we probably should only say the the constrained version follows the same semantics as the "regular" LLVM intrinsics.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71218/new/

https://reviews.llvm.org/D71218





More information about the llvm-commits mailing list