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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 28 19:16:45 PDT 2020


arsenm added a comment.
Herald added subscribers: jdoerfert, wuzish.
Herald added a reviewer: jdoerfert.

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?


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