Remove surface state generation in patch tokens for printf buffer

Printf buffer is always addressed with stateless addressing mode, so
surface state is useless

Author: aratajew

IGC/AdaptorOCL/OCL/sp/sp_g8.cpp

@@ -788,21 +788,6 @@ RETVAL CGen8OpenCLStateProcessor::CreateSurfaceStateHeap(
         }
     }
 
-    if (annotations.m_printfBufferAnnotation != NULL)
-    {
-        unsigned int bti = annotations.m_argIndexMap.at(annotations.m_printfBufferAnnotation->ArgumentNumber);
-        context.Surface.SurfaceOffset[bti] = (DWORD)membuf.Size();
-
-        SurfaceStates.insert(
-            std::make_pair(
-                bti,
-                SurfaceState(
-                    SURFACE_BUFFER,
-                    SURFACE_FORMAT_RAW,
-                    0,
-                    false)));
-    }
-
     if (annotations.m_syncBufferAnnotation != NULL)
     {
         unsigned int bti = annotations.m_argIndexMap.at(annotations.m_syncBufferAnnotation->ArgumentNumber);
@@ -1653,12 +1638,10 @@ RETVAL CGen8OpenCLStateProcessor::CreatePatchList(
             iOpenCL::SPatchAllocateStatelessPrintfSurface  patch;
             memset(&patch, 0, sizeof(patch));
 
-            unsigned int bti = annotations.m_argIndexMap.at(printfBufAnn->ArgumentNumber);
-
             patch.Token = iOpenCL::PATCH_TOKEN_ALLOCATE_STATELESS_PRINTF_SURFACE;
             patch.Size = sizeof(patch);
             patch.PrintfSurfaceIndex = printfBufAnn->Index;
-            patch.SurfaceStateHeapOffset = context.Surface.SurfaceOffset[bti];
+            patch.SurfaceStateHeapOffset = UINT32_MAX;
             patch.DataParamOffset = printfBufAnn->PayloadPosition;
             patch.DataParamSize = printfBufAnn->DataSize;
 

IGC/Compiler/CISACodeGen/OpenCLKernelCodeGen.cpp

@@ -1495,13 +1495,9 @@ namespace IGC
 
         case KernelArg::ArgType::IMPLICIT_PRINTF_BUFFER:
         {
-            int argNo = kernelArg->getAssociatedArgNo();
-            SOpenCLKernelInfo::SResourceInfo resInfo = getResourceInfo(argNo);
-            m_kernelInfo.m_argIndexMap[argNo] = getBTI(resInfo);
-
             auto printfBuffer = std::make_unique<iOpenCL::PrintfBufferAnnotation>();
 
-            printfBuffer->ArgumentNumber = argNo;
+            printfBuffer->ArgumentNumber = kernelArg->getAssociatedArgNo();
             printfBuffer->PayloadPosition = payloadPosition;
             printfBuffer->DataSize = kernelArg->getAllocateSize();
             printfBuffer->Index = 0; // This value is not used by Runtime.

IGC/Compiler/Optimizer/OpenCLPasses/ResourceAllocator/ResourceAllocator.cpp

@@ -158,7 +158,6 @@ static AllocationType getAllocationType(KernelArg::ArgType argType, BindlessAllo
     case KernelArg::ArgType::IMPLICIT_CONSTANT_BASE:
     case KernelArg::ArgType::IMPLICIT_GLOBAL_BASE:
     case KernelArg::ArgType::IMPLICIT_PRIVATE_BASE:
-    case KernelArg::ArgType::IMPLICIT_PRINTF_BUFFER:
     case KernelArg::ArgType::IMPLICIT_SYNC_BUFFER:
     case KernelArg::ArgType::IMPLICIT_RT_GLOBAL_BUFFER:
     case KernelArg::ArgType::IMPLICIT_DEVICE_ENQUEUE_EVENT_POOL: