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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

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

> In D84551#2173159 <https://reviews.llvm.org/D84551#2173159>, @arsenm wrote:
>
> > This will conflict with D84159 <https://reviews.llvm.org/D84159>.
> >
> > Why does the IR have rint, lrint, and llrint? libm has all 3, but they seem to just differ by the return integer type? Why isn't there just the one rint intrinsic and clang is responsible for picking the right result type?
>
>
> Conflict in what way?


Touches the same lines


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84551