[PATCH] D68810: Document rounding for llvm.lround and llvm.lrint

[Kevin P. Neal via Phabricator via llvm-commits]

kpn marked an inline comment as done.
kpn added inline comments.


================
Comment at: llvm/docs/LangRef.rst:12788
 
-The '``llvm.lrint.*``' intrinsics returns the operand rounded to the
-nearest integer.
+The '``llvm.lrint.*``' intrinsics return the operand rounded to the nearest
+integer.
----------------
efriedma wrote:
> craig.topper wrote:
> > Doesn't lrint use the current rounding mode?
> Sort of?  I mean, in this context, we assume the "current rounding mode" is round-to-nearest-even.
> 
> Probably worth clarifying what we mean by "nearest" here.
We specify how rounding is done very rarely, and when we do we usually say "nearest integer" without mentioning the rounding mode.

Do we want to change all references to "nearest integer" to state that the current rounding mode is used? Do we want a mention somewhere else to cover all uses?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68810