[IGC vISA] Fix the issue that mul QDD is not supported

Fix the issue that mul QDD is not supported on PVCXT
intel · Aug 29, 2023 · bef6955 · bef6955
1 parent 70a1401
commit bef6955
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/visa/HWCaps.inc b/visa/HWCaps.inc
@@ -345,6 +345,12 @@ bool noFP64() const {
   return getPlatform() == GENX_ICLLP || isXeLP() || getPlatform() == Xe_DG2;
 }
 
+bool hasQDDMul() const {
+  return getPlatform() == GENX_BDW || getPlatform() == GENX_SKL ||
+         getPlatform() == GENX_BXT || getPlatform() == GENX_CNL ||
+         getPlatform() == Xe_XeHPSDV || getPlatform() == Xe_PVC;
+}
+
 bool no64bitRegioning() const {
   return getPlatform() == GENX_CHV || getPlatform() == GENX_BXT ||
          getPlatform() == GENX_ICLLP || isXeLP() || getPlatform() == Xe_DG2 ||

diff --git a/visa/HWConformity.cpp b/visa/HWConformity.cpp
@@ -2396,7 +2396,7 @@ bool HWConformity::fixMULInst(INST_LIST_ITER &i, G4_BB *bb) {
   }
 
   bool doNativeMul = false;
-  if (!builder.no64bitRegioning()) {
+  if (builder.hasQDDMul()) {
     // platform natively supports DW-DW multiply, no need to generate
     // mul/mach/mov sequence
     doNativeMul = true;
@@ -2673,7 +2673,7 @@ void HWConformity::fixMULHInst(INST_LIST_ITER &i, G4_BB *bb) {
   }
 
   bool useMulQDD = false;
-  if (execSize <= builder.getNativeExecSize() && !builder.no64bitRegioning() &&
+  if (execSize <= builder.getNativeExecSize() && builder.hasQDDMul() &&
       builder.supportFloatOr64bRegioning()) {
     useMulQDD = true;
     if (!IS_DTYPE(src0->getType()) || !IS_DTYPE(src1->getType())) {