[PATCH] D84551: [GlobalISel] Add G_INTRINSIC_LRINT and translate from llvm.lrint

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 09:43:01 PDT 2020


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

In D84551#2182412 <https://reviews.llvm.org/D84551#2182412>, @aemerson wrote:

> Ping

These intrinsics were added a lot more recently than I thought by D62017 <https://reviews.llvm.org/D62017>

I think it doesn't really make sense to provide these redundant intrinsics, and math intrinsics are not supposed to provide a 1:1 wrapper around libm. But they're there now, and I would like to eliminate the intrinsic handling delta against SelectionDAG


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84551



More information about the llvm-commits mailing list