Remove igc metrics

Remove igc metrics

Author: lwesiers

IGC/AdaptorOCL/UnifyIROCL.cpp

@@ -136,8 +136,6 @@ SPDX-License-Identifier: MIT
 
 #include <string>
 
-#include <Metrics/IGCMetric.h>
-
 using namespace llvm;
 using namespace IGC::IGCMD;
 using namespace IGC::Debug;
@@ -185,9 +183,6 @@ static void CommonOCLBasedPasses(OpenCLProgramContext *pContext) {
 
   setupTriple(*pContext);
 
-  pContext->metrics.Init(&pContext->hash, pContext->getModule()->getNamedMetadata("llvm.dbg.cu") != nullptr);
-  pContext->metrics.CollectFunctions(pContext->getModule());
-
   unify_opt_PreProcess(pContext);
   pContext->m_checkFastFlagPerInstructionInCustomUnsafeOptPass = true;
   pContext->m_mayHaveUnalignedAddressRegister = true;

IGC/AdaptorOCL/dllInterfaceCompute.cpp

@@ -135,8 +135,6 @@ SPDX-License-Identifier: MIT
 #include <llvm/Support/SourceMgr.h>
 #include "common/LLVMWarningsPop.hpp"
 
-#include "IGC/Metrics/IGCMetric.h"
-
 using namespace IGC::IGCMD;
 using namespace IGC::Debug;
 using namespace IGC;
@@ -1157,7 +1155,6 @@ bool TranslateBuildSPMD(const STB_TranslateInputArgs *pInputArgs, STB_TranslateO
     compOpt->FloatDenormModeBFTF = FLOAT_DENORM_RETAIN;
   }
 
-  unsigned PtrSzInBits = pKernelModule->getDataLayout().getPointerSizeInBits();
   // TODO: Again, this should not happen on each compilation
 
   bool doSplitModule = oclContext.m_InternalOptions.CompileOneKernelAtTime || IGC_IS_FLAG_ENABLED(CompileOneAtTime);
@@ -1312,17 +1309,11 @@ bool TranslateBuildSPMD(const STB_TranslateInputArgs *pInputArgs, STB_TranslateO
     SetOutputMessage(oclContext.GetWarning(), *pOutputArgs);
   }
 
-  // Prepare and set program binary
-  unsigned int pointerSizeInBytes = (PtrSzInBits == 64) ? 8 : 4;
-
   // FIXME: zebin currently only support program output itself, will add debug info
   // into it
   size_t binarySize = 0;
   char *binaryOutput = nullptr;
 
-  oclContext.metrics.FinalizeStats();
-  oclContext.metrics.OutputMetrics();
-
   llvm::SmallVector<char, 64> buf;
   llvm::raw_svector_ostream llvm_os(buf);
   const bool excludeIRFromZEBinary =
@@ -1334,12 +1325,14 @@ bool TranslateBuildSPMD(const STB_TranslateInputArgs *pInputArgs, STB_TranslateO
     spv_size = pInputArgs->InputSize;
   }
 
-  // IGC metrics
-  size_t metricDataSize = oclContext.metrics.getMetricDataSize();
-  auto metricData = reinterpret_cast<const char *>(oclContext.metrics.getMetricData());
+  // IGC metrics are empty
+  auto metricData = "n\a";
+  size_t metricDataSize = sizeof(metricData);
 
+  unsigned PtrSzInBits = pKernelModule->getDataLayout().getPointerSizeInBits();
+  unsigned int pointerSizeInBytes = (PtrSzInBits == 64) ? 8 : 4;
   oclContext.m_programOutput.GetZEBinary(llvm_os, pointerSizeInBytes, spv_data, spv_size, metricData, metricDataSize,
-                                         pInputArgs->pOptions, pInputArgs->OptionsSize);
+    pInputArgs->pOptions, pInputArgs->OptionsSize);
 
   // FIXME: try to avoid memory copy here
   binarySize = buf.size();

IGC/CMakeLists.txt

@@ -1533,9 +1533,6 @@ igc_sg_define(IGC__common)
 igc_sg_define(IGC__Common_CLElfLib)
 
 
-# Add protobuf
-include(igc_find_protobuf)
-
 set(IGC_BUILD__SPIRV_TOOLS_ENABLED ON)
 
 if(IGC_BUILD__SPIRV_TOOLS_ENABLED)
@@ -1636,9 +1633,6 @@ if(MSVC)
   )
 endif()
 
-#Metrcis
-include(Metrics/protobuf_metrics.cmake)
-
 # Sources/headers for main libraries.
 set(IGC_BUILD__SRC__IGC__igc_common
     ${IGC_BUILD__SRC__IGC_AdaptorOCL}
@@ -1902,10 +1896,6 @@ foreach(_libBuildSuffix ${IGC_BUILD__MAIN_IGC_LIB_SUFFIXES})
     )
   endif()
 
-  #Metrics
-  list(APPEND _targetLinkLineCommon
-      igc_metric
-  )
 
 list(APPEND _targetLinkLineCommon zebinlib)
 list(APPEND _targetLinkLineCommon IGCOptions)

IGC/Compiler/CISACodeGen/CISABuilder.cpp

@@ -3655,11 +3655,6 @@ void CEncoder::InitVISABuilderOptions(TARGET_PLATFORM VISAPlatform, bool canAbor
     SaveOption(vISA_GenerateDebugInfo, true);
   }
 
-  if (context->metrics.Enable()) {
-    SaveOption(vISA_GenerateKernelInfo, true);
-    SaveOption(vISA_EmitLocation, true);
-  }
-
   if (canAbortOnSpill) {
     SaveOption(vISA_AbortOnSpill, true);
   }
@@ -5405,9 +5400,6 @@ void CEncoder::Compile(bool hasSymbolTable, GenXFunctionGroupAnalysis *&pFGA) {
 
   KERNEL_INFO *vISAstats;
   pMainKernel->GetKernelInfo(vISAstats);
-  // Collect metrics from vISA
-  context->metrics.CollectRegStats(vISAstats, m_program->entry);
-
   // Depend on vISA information about barriers presence to make sure that it's
   // always set properly, even if a barrier is used as a part of Inline vISA
   // code only.

IGC/Compiler/CISACodeGen/CheckInstrTypes.cpp

@@ -89,8 +89,6 @@ bool CheckInstrTypes::runOnFunction(Function &F) {
   context = getAnalysis<CodeGenContextWrapper>().getCodeGenContext();
 
   LI = &getAnalysis<LoopInfoWrapperPass>().getLoopInfo();
-  if (g_metrics)
-    context->metrics.CollectLoops(LI);
 
   // check if module has debug info
   g_InstrTypes.hasDebugInfo = F.getParent()->getNamedMetadata("llvm.dbg.cu") != nullptr;

IGC/Compiler/CISACodeGen/DebugInfo.cpp

@@ -238,9 +238,6 @@ bool DebugInfoPass::runOnModule(llvm::Module &M) {
     currShader->ProgramOutput()->m_debugDataGenISASize = 0;
     currShader->ProgramOutput()->m_debugDataGenISA = nullptr;
 
-    m_currShader->GetContext()->metrics.CollectDataFromDebugInfo(m_currShader->entry, &m_currShader->GetDebugInfoData(),
-                                                                 &VisaDbgInfo);
-
     if (finalize) {
       IDebugEmitter::Release(m_pDebugEmitter);
     }

IGC/Compiler/CISACodeGen/Emu64OpsPass.cpp

@@ -578,12 +578,10 @@ bool Emu64Ops::expandInsts(Function &F) {
     for (auto BI = BB->begin(), BE = BB->end(); BI != BE; /*EMPTY*/) {
       Instruction *I = &(*BI++);
       doNotDeleteInstr = false;
-      CGC->metrics.StatBeginEmuFunc(I);
       bool LocalChanged = Expander->expand(I);
       Changed |= LocalChanged;
 
       if (LocalChanged) {
-        CGC->metrics.StatEndEmuFunc(I);
         BI = std::next(BasicBlock::iterator(I));
         BE = I->getParent()->end();
         DeadInsts.insert(I);

IGC/Compiler/CISACodeGen/LowerGEPForPrivMem.cpp

@@ -724,9 +724,6 @@ void LowerGEPForPrivMem::visitAllocaInst(AllocaInst &I) {
   IGC_ASSERT(I.getType()->getAddressSpace() == ADDRESS_SPACE_PRIVATE);
 
   StatusPrivArr2Reg status = CheckIfAllocaPromotable(&I);
-  if (I.getType()->getAddressSpace() == ADDRESS_SPACE_PRIVATE) {
-    m_ctx->metrics.CollectMem2Reg(&I, status);
-  }
   if (status != StatusPrivArr2Reg::OK) {
     MarkNotPromtedAllocas(I, status);
     // alloca size extends remain per-lane-reg space

IGC/Compiler/CISACodeGen/PartialEmuI64OpsPass.cpp

@@ -708,12 +708,10 @@ bool PartialEmuI64Ops::expandInsts(Function &F) {
   for (auto &BB : RPOT) {
     for (auto BI = BB->begin(), BE = BB->end(); BI != BE; /*EMPTY*/) {
       Instruction *I = &(*BI++);
-      CGC->metrics.StatBeginEmuFunc(I);
       bool LocalChanged = Expander->expand(I);
       Changed |= LocalChanged;
 
       if (LocalChanged) {
-        CGC->metrics.StatEndEmuFunc(I);
         BI = std::next(BasicBlock::iterator(I));
         BE = I->getParent()->end();
         DeadInsts.insert(I);

IGC/Compiler/CISACodeGen/PatternMatchPass.cpp

@@ -1262,11 +1262,6 @@ void CodeGenPatternMatch::visitCallInst(CallInst &I) {
   } else {
     Function *Callee = I.getCalledFunction();
 
-    if (IGCMetrics::IGCMetric::isMetricFuncCall(&I)) {
-      // dont do anything with metrics calls
-      return;
-    }
-
     // Match inline asm
     if (I.isInlineAsm()) {
       if (getAnalysis<CodeGenContextWrapper>().getCodeGenContext()->m_DriverInfo.SupportInlineAssembly()) {