[PATCH] D50401: [GISel]: Add Opcodes for a few Libm Intrinsics

Aditya Nandakumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 10 10:54:51 PDT 2018


aditya_nandakumar added inline comments.


================
Comment at: include/llvm/Target/GenericOpcodes.td:516
+//------------------------------------------------------------------------------
+// LIBM Intrinsics
+//------------------------------------------------------------------------------
----------------
aemerson wrote:
> Perhaps say "LIBM compatible intrinsics"
Would you name it G_FTRUNC? I thought that is very similar to G_FPTRUNC and might lead to confusion. One other name I can think of is G_INTRINSIC_TRUNC which makes it explicit that this is the intrinsic equivalent in llvm IR. 


Repository:
  rL LLVM

https://reviews.llvm.org/D50401





More information about the llvm-commits mailing list