Use int copy instead of float copy during spill code generation.

Change-Id: Ibfe337ecde398ef55ba90c6757507ef870a8dc60

Author: weiyu-chen

visa/G4_Opcode.h

@@ -421,6 +421,25 @@ inline bool isLowPrecisionFloatTy(G4_Type ty)
     return ty == Type_HF;
 }
 
+inline G4_Type floatToSameWidthIntType(G4_Type floatTy)
+{
+    assert(IS_TYPE_FLOAT_ALL(floatTy));
+    switch (getTypeSize(floatTy))
+    {
+    case 1:
+        return Type_UB;
+    case 2:
+        return Type_UW;
+    case 4:
+        return Type_UD;
+    case 8:
+        return Type_UQ;
+    default:
+        assert(false && "illegal type size");
+        return Type_UD;
+    }
+}
+
 #define GRFALIGN (Sixteen_Word)
 #define HALFGRFALIGN (Eight_Word)
 #endif  // _G4_OPCODE_H_

visa/SpillManagerGMRF.cpp

@@ -3260,9 +3260,17 @@ SpillManagerGRF::insertSpillRangeCode (
                 ((*spilledInstIter)->opcode() != G4_sel)?
                 (*spilledInstIter)->getPredicate () : nullptr;
 
+            if (tmpRangeSrcRegion->getType() == spillRangeDstRegion->getType() && IS_TYPE_FLOAT_ALL(tmpRangeSrcRegion->getType()))
+            {
+                // use int copy when possible as floating-point copy moves may need further legalization
+                auto equivIntTy = floatToSameWidthIntType(tmpRangeSrcRegion->getType());
+                tmpRangeSrcRegion->setType(equivIntTy);
+                spillRangeDstRegion->setType(equivIntTy);
+            }
+
             createMovInst (
                 execSize, spillRangeDstRegion, tmpRangeSrcRegion,
-                predicate != nullptr ? builder_->duplicateOperand(predicate) : predicate,
+                builder_->duplicateOperand(predicate),
                 (*spilledInstIter)->getMaskOption());
             numGRFMove ++;