Minor refactor on accumulator legality check.

weiyu-chen · sys_zuul · commit aaf8589eaf05 · 2020-07-16T20:22:56.000-07:00
Change-Id: Ib348bb85d7108ba81a7a3e50bec3f86d039d1104
diff --git a/visa/Gen4_IR.cpp b/visa/Gen4_IR.cpp
@@ -7764,6 +7764,11 @@ bool G4_INST::canDstBeAcc() const
         return false;
     }
 
+    if (!builder.hasFP64Acc() && dst->getType() == Type_DF)
+    {
+        return false;
+    }
+
     // src0 may not have indirect regioning
     if (!builder.accDstforIndirectSrc() && getSrc(0) && getSrc(0)->isSrcRegRegion())
     {
diff --git a/visa/HWCaps.inc b/visa/HWCaps.inc
@@ -627,5 +627,10 @@
         return false;
     }
 
+    bool hasFP64Acc() const
+    {
+        return !noFP64();
+    }
+
 
     // end HW capabilities

Original file line number	Diff line number	Diff line change
`@@ -7764,6 +7764,11 @@ bool G4_INST::canDstBeAcc() const`
`7764`	`7764`	`return false;`
`7765`	`7765`	`}`
`7766`	`7766`
	`7767`	`+ if (!builder.hasFP64Acc() && dst->getType() == Type_DF)`
	`7768`	`+ {`
	`7769`	`+ return false;`
	`7770`	`+ }`
	`7771`	`+`
`7767`	`7772`	`// src0 may not have indirect regioning`
`7768`	`7773`	`if (!builder.accDstforIndirectSrc() && getSrc(0) && getSrc(0)->isSrcRegRegion())`
`7769`	`7774`	`{`