[PATCH] D75670: [FPEnv] Intrinsic llvm.roundeven

Thu Mar 5 13:12:30 PST 2020

efriedma added inline comments.

================
Comment at: llvm/docs/LangRef.rst:12931-12934
+'``llvm.roundeven.*``' Intrinsic
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+Syntax:
----------------
arsenm wrote:
> efriedma wrote:
> > arsenm wrote:
> > > Isn't this the same as rint? Since the default rounding mode is assumed 
> > The name is a hint that we should prefer to lower to a library function named "roundeven", if we can't generate an inline sequence.  Other than that, yes, it's identical.
> > 
> > I vaguely recall that I tried to merge llvm.rint and llvm.nearbyint at some point, but someone ran into issues with missing symbols.  I can try to dig it up if you think it's important.
> I don't think the intrinsic name should ever influence codegen that way. We should not have redundant intrinsics, especially for some weird lowering hint
See https://reviews.llvm.org/rL299247 for recent history of this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75670