Mark indirect calls uniform

When indirect calls are uniform, VISA can avoid applying some SW WAs (eg, fused call WA = 2).
intel · Aug 30, 2023 · 7ab2f49 · 7ab2f49
1 parent 48650b1
commit 7ab2f49
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/IGC/Compiler/CISACodeGen/CISABuilder.cpp b/IGC/Compiler/CISACodeGen/CISABuilder.cpp
@@ -417,7 +417,8 @@ namespace IGC
         VISA_Exec_Size execSize = visaExecSize(m_program->m_dispatchSize);
         VISA_VectorOpnd* funcAddrOpnd = GetSourceOperandNoModifier(funcPtr);
         V(vKernel->AppendVISACFIndirectFuncCallInst(
-            predOpnd, emask, execSize, false, funcAddrOpnd, argSize, retSize));
+            predOpnd, emask, execSize, funcPtr->IsUniform(), funcAddrOpnd,
+            argSize, retSize));
     }
 
     void CEncoder::SubroutineRet(CVariable* flag, llvm::Function* F)

diff --git a/IGC/VectorCompiler/lib/GenXCodeGen/GenXCisaBuilder.cpp b/IGC/VectorCompiler/lib/GenXCodeGen/GenXCisaBuilder.cpp
@@ -867,7 +867,7 @@ class GenXKernelBuilder {
                                                unsigned char ReturnSize) {
     updateSIMDSize(ExecMask, ExecSize);
     CISA_CALL(Kernel->AppendVISACFIndirectFuncCallInst(
-        Pred, ExecMask, ExecSize, false, FuncAddr, ArgSize, ReturnSize));
+        Pred, ExecMask, ExecSize, true, FuncAddr, ArgSize, ReturnSize));
   }
 
   inline void appendVISACFRetInst(VISA_PredOpnd *Pred, VISA_EMask_Ctrl ExecMask,