[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