(2nd try) make sure we split all SIMD16 mix mode instructions, not just the ones with HF/F destination

weiyu-chen · sys_zuul · commit 6e1c55c68e46 · 2019-10-07T14:35:30.000-07:00
Change-Id: I68e4effb91f012b97a7c1f811b5d5ed7270631b7
diff --git a/visa/HWConformity.cpp b/visa/HWConformity.cpp
@@ -7385,11 +7385,7 @@ void HWConformity::fixMixedHFInst(G4_BB* bb)
         if (getGenxPlatform() >= GENX_CHV)
         {
             // no SIMD16 mix mode instruction
-            dst = inst->getDst();
-            bool isDstPackedHForF =
-                (dst->getType() == builder.getMixModeType() && dst->getHorzStride() == 1) ||
-                dst->getType() == Type_F;
-            if (inst->getExecSize() > 8 && isDstPackedHForF)
+            if (inst->getExecSize() > builder.getNativeExecSize() && inst->isMixedMode())
             {
                 evenlySplitInst(instIter, bb, false);
                 //instruction was split, and new instruction inserted before

Original file line number	Diff line number	Diff line change
`@@ -7385,11 +7385,7 @@ void HWConformity::fixMixedHFInst(G4_BB* bb)`
`7385`	`7385`	`if (getGenxPlatform() >= GENX_CHV)`
`7386`	`7386`	`{`
`7387`	`7387`	`// no SIMD16 mix mode instruction`
`7388`		`- dst = inst->getDst();`
`7389`		`- bool isDstPackedHForF =`
`7390`		`- (dst->getType() == builder.getMixModeType() && dst->getHorzStride() == 1) \|\|`
`7391`		`- dst->getType() == Type_F;`
`7392`		`- if (inst->getExecSize() > 8 && isDstPackedHForF)`
	`7388`	`+ if (inst->getExecSize() > builder.getNativeExecSize() && inst->isMixedMode())`
`7393`	`7389`	`{`
`7394`	`7390`	`evenlySplitInst(instIter, bb, false);`
`7395`	`7391`	`//instruction was split, and new instruction inserted before`