Moving V->HF causes incorrect results. Don't copyprog V to

jgu222 · gfxbot · commit 9c5302c33efb · 2019-04-08T02:56:39.000-07:00
avoid V-&gt;HF mov

Change-Id: I2f5472b8fb9630d53d3a2d96e3893c352ce98980
diff --git a/visa/Gen4_IR.cpp b/visa/Gen4_IR.cpp
@@ -1670,6 +1670,14 @@ bool G4_INST::isLegalType(G4_Type type, Gen4_Operand_Number opndNum) const
         return type == Type_B || type == Type_UB;
     case G4_subb:
         return type == Type_UD;
+    case G4_mov:
+        // Avoid mov r7.0<1>:hf  0x76543210:v
+        if (IS_VINTTYPE(type) &&
+            (IS_FTYPE(dst->getType()) || IS_HFTYPE(dst->getType())))
+        {
+            return false;
+        }
+        return true;
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1670,6 +1670,14 @@ bool G4_INST::isLegalType(G4_Type type, Gen4_Operand_Number opndNum) const`
`1670`	`1670`	`return type == Type_B \|\| type == Type_UB;`
`1671`	`1671`	`case G4_subb:`
`1672`	`1672`	`return type == Type_UD;`
	`1673`	`+ case G4_mov:`
	`1674`	`+ // Avoid mov r7.0<1>:hf 0x76543210:v`
	`1675`	`+ if (IS_VINTTYPE(type) &&`
	`1676`	`+ (IS_FTYPE(dst->getType()) \|\| IS_HFTYPE(dst->getType())))`
	`1677`	`+ {`
	`1678`	`+ return false;`
	`1679`	`+ }`
	`1680`	`+ return true;`
`1673`	`1681`	`}`
`1674`	`1682`	`}`
`1675`	`1683`