Remove ConstBase and GlobalBase implicit arguments for PVC+ platforms

A second try (recommit) of a8cc7c1.
This reverts commit 0a5a1c0.

Author: pszymich

IGC/Compiler/Optimizer/OpenCLPasses/ProgramScopeConstants/ProgramScopeConstantAnalysis.cpp

@@ -264,12 +264,17 @@ bool ProgramScopeConstantAnalysis::runOnModule(Module& M)
     // Stackcalls:
     //  Stackcall ABI does not allow implicit args, so rely on relocation for global variable access
 
+    // Disable ConstBase and GlobalBase implicit arguments on PVC+.
+    // StatelessToStateful pass is not enabled for most of the cases so there
+    // is no benefit to add the implicit arguments. Const/Global variables
+    // access can just go through relocations.
     // Workaround: When there is stringConstants in the module, do not insert
     // implicit arguments to prevent const vars getting promoted
     // at statelessToStateful pass. In zebin path, stateful promotion
     // of const vars can't work well with printf strings.
-    bool skipConstAndGlobalBaseArgs =
-             Ctx->enableZEBinary() && !m_pModuleMd->stringConstants.empty();
+    bool skipConstAndGlobalBaseArgs = Ctx->enableZEBinary() &&
+         (Ctx->platform.GetProductFamily() >= IGFX_PVC ||
+          !m_pModuleMd->stringConstants.empty());
 
     if (!skipConstAndGlobalBaseArgs && hasInlineConstantBuffer)
     {