[PATCH] D62017: [CodeGen] Add lrint/llrint builtins

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D62017#2180847 <https://reviews.llvm.org/D62017#2180847>, @arsenm wrote:

> What is the point of these intrinsics exactly? It seems like it's just copying the libm function names, despite them only differing in the return integer type. Why can't this be accomplished with adding type mangling to the regular rint intrinsic?

I think the primary complication is that we need to know what libcall to emit if the target doesn't natively support them. The backend doesn't know how to map from IR type "long" or "long long" to generate the right libcall.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62017