[PATCH] D62018: [AArch64] Handle ISD::LRINT and ISD::LLRINT

[Adhemerval Zanella via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.td:3159
 
+def : Pat<(i64 (lrint f32:$Rn)),
+          (FCVTZSUXSr (!cast<Instruction>(FRINTXSr) f32:$Rn))>;
----------------
SjoerdMeijer wrote:
> mstorsjo wrote:
> > Please add patterns for returning `i32` as well; the original C standard math function, which is turned into the llvm IR intrinsic, returns `long`, which is 32 bit on windows platforms. See D62108 for a fix for the same for lround.
> And also a pattern for (lrint f16)?
I plan to address lrint and lround for f16 in a subsequent patch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62018