AMDGPU: Update isFPImmLegal for f16
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Thu, 22 Dec 2016 03:05:30 +0000 (03:05 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Thu, 22 Dec 2016 03:05:30 +0000 (03:05 +0000)
I don't think this matters because ConstantFP is legal.

llvm-svn: 290299

llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp

index 7884c4c..cd0d60f 100644 (file)
@@ -498,7 +498,8 @@ bool AMDGPUTargetLowering::isSelectSupported(SelectSupportKind SelType) const {
 // FIXME: Why are we reporting vectors of FP immediates as legal?
 bool AMDGPUTargetLowering::isFPImmLegal(const APFloat &Imm, EVT VT) const {
   EVT ScalarVT = VT.getScalarType();
-  return (ScalarVT == MVT::f32 || ScalarVT == MVT::f64);
+  return (ScalarVT == MVT::f32 || ScalarVT == MVT::f64 ||
+         (ScalarVT == MVT::f16 && Subtarget->has16BitInsts()));
 }
 
 // We don't want to shrink f64 / f32 constants.