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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: include/llvm/Target/GenericOpcodes.td:516
+//------------------------------------------------------------------------------
+// LIBM Intrinsics
+//------------------------------------------------------------------------------
----------------
aditya_nandakumar wrote:
> 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. 
Yes


Repository:
  rL LLVM

https://reviews.llvm.org/D50401