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

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


arsenm added a comment.

In D62017#2180861 <https://reviews.llvm.org/D62017#2180861>, @craig.topper wrote:

> so that still needs the ISD node to carry the function name to reverse back to library call name. The mapping from IR type to long vs long long is target and OS dependent.

I don't understand why you can't derive the call name from the triple


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62017



More information about the llvm-commits mailing list