[PATCH] D26767: [AMDGPU] Promote f16/i16 conversions to f32/i32 + custom lower f16 = fp_round f64

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 13:53:07 PST 2016


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:3664-3668
+  case ISD::BITCAST: {
+    if (N->getValueType(0) == N->getOperand(0).getValueType())
+      return N->getOperand(0);
+    break;
+  }
----------------
kzhuravl wrote:
> arsenm wrote:
> > This should be very unnecessary
> I am getting a "cannot select: i16 = bitcast i16". Do you have a suggestion on how to solve it?
> 
> Thanks
Where is that coming from? I thought getNode folded out trivial bitcasts like this already


https://reviews.llvm.org/D26767





More information about the llvm-commits mailing list