Refactor GenXOpts pass-names

.

Author: igorban-intel

IGC/VectorCompiler/lib/GenXOpts/CMAnalysis/InstructionSimplifyGenX.cpp

@@ -44,7 +44,7 @@ SPDX-License-Identifier: MIT
 
 #include "Probe/Assertion.h"
 
-#define DEBUG_TYPE "genx-simplify"
+#define DEBUG_TYPE "GenXSimplify"
 
 using namespace llvm;
 
@@ -505,10 +505,10 @@ bool GenXSimplify::runOnFunction(Function &F) {
 }
 
 char GenXSimplify::ID = 0;
-INITIALIZE_PASS_BEGIN(GenXSimplify, "genx-simplify",
+INITIALIZE_PASS_BEGIN(GenXSimplify, "GenXSimplify",
                       "simplify genx specific instructions", false, false)
 INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
-INITIALIZE_PASS_END(GenXSimplify, "genx-simplify",
+INITIALIZE_PASS_END(GenXSimplify, "GenXSimplify",
                     "simplify genx specific instructions", false, false)
 
 FunctionPass *llvm::createGenXSimplifyPass() { return new GenXSimplify; }

IGC/VectorCompiler/lib/GenXOpts/CMTrans/CMABI.cpp

@@ -84,7 +84,7 @@ SPDX-License-Identifier: MIT
 #include <unordered_set>
 #include <vector>
 
-#define DEBUG_TYPE "cmabi"
+#define DEBUG_TYPE "CMABI"
 
 using namespace llvm;
 
@@ -317,10 +317,10 @@ struct CMABI : public CallGraphSCCPass {
 } // namespace
 
 char CMABIAnalysis::ID = 0;
-INITIALIZE_PASS_BEGIN(CMABIAnalysis, "cmabi-analysis",
+INITIALIZE_PASS_BEGIN(CMABIAnalysis, "CMABIAnalysis",
                       "helper analysis pass to get info for CMABI", false, true)
 INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
-INITIALIZE_PASS_END(CMABIAnalysis, "cmabi-analysis",
+INITIALIZE_PASS_END(CMABIAnalysis, "CMABIAnalysis",
                     "Fix ABI issues for the genx backend", false, true)
 
 bool CMABIAnalysis::runOnModule(Module &M) {
@@ -936,10 +936,12 @@ void CMABI::diagnoseOverlappingArgs(CallInst *CI)
 }
 
 char CMABI::ID = 0;
-INITIALIZE_PASS_BEGIN(CMABI, "cmabi", "Fix ABI issues for the genx backend", false, false)
+INITIALIZE_PASS_BEGIN(CMABI, "CMABI", "Fix ABI issues for the genx backend",
+                      false, false)
 INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
 INITIALIZE_PASS_DEPENDENCY(CMABIAnalysis)
-INITIALIZE_PASS_END(CMABI, "cmabi", "Fix ABI issues for the genx backend", false, false)
+INITIALIZE_PASS_END(CMABI, "CMABI", "Fix ABI issues for the genx backend",
+                    false, false)
 
 Pass *llvm::createCMABIPass() { return new CMABI(); }
 
@@ -1005,13 +1007,13 @@ struct CMLowerVLoadVStore : public FunctionPass {
 
 char CMLowerVLoadVStore::ID = 0;
 INITIALIZE_PASS_BEGIN(CMLowerVLoadVStore, "CMLowerVLoadVStore",
-                      "Lower CM reference vector loads and stores", false, false)
+                      "Lower CM reference vector loads and stores", false,
+                      false)
 INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
 INITIALIZE_PASS_DEPENDENCY(PostDominatorTreeWrapperPass)
 INITIALIZE_PASS_END(CMLowerVLoadVStore, "CMLowerVLoadVStore",
                     "Lower CM reference vector loads and stores", false, false)
 
-
 bool CMLowerVLoadVStore::runOnFunction(Function &F) {
   bool Changed = false;
   Changed |= promoteAllocas(F);

IGC/VectorCompiler/lib/GenXOpts/CMTrans/CMImpParam.cpp

@@ -100,7 +100,7 @@ SPDX-License-Identifier: MIT
 ///
 //===----------------------------------------------------------------------===//
 
-#define DEBUG_TYPE "cmimpparam"
+#define DEBUG_TYPE "CMImpParam"
 
 #include "vc/GenXOpts/GenXOpts.h"
 #include "vc/Utils/GenX/KernelInfo.h"
@@ -1198,14 +1198,16 @@ CMImpParam::processKernelParameters(Function *F,
 }
 
 char CMImpParam::ID = 0;
-INITIALIZE_PASS_BEGIN(CMImpParam, "cmimpparam",
+INITIALIZE_PASS_BEGIN(CMImpParam, "CMImpParam",
                       "Transformations required to support implicit arguments",
                       false, false)
 INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
-INITIALIZE_PASS_END(CMImpParam, "cmimpparam",
+INITIALIZE_PASS_END(CMImpParam, "CMImpParam",
                     "Transformations required to support implicit arguments",
                     false, false)
 
-Pass *llvm::createCMImpParamPass(bool HasPayloadInMemory) {
+namespace llvm {
+Pass *createCMImpParamPass(bool HasPayloadInMemory) {
   return new CMImpParam{HasPayloadInMemory};
 }
+} // namespace llvm

IGC/VectorCompiler/lib/GenXOpts/CMTrans/CMKernelArgOffset.cpp

@@ -84,8 +84,6 @@ SPDX-License-Identifier: MIT
 ///
 //===----------------------------------------------------------------------===//
 
-#define DEBUG_TYPE "cmkernelargoffset"
-
 #include <llvmWrapper/IR/Type.h>
 #include "llvmWrapper/Support/Alignment.h"
 
@@ -107,6 +105,8 @@ SPDX-License-Identifier: MIT
 
 #include "Probe/Assertion.h"
 
+#define DEBUG_TYPE "CMKernelArgOffset"
+
 using namespace llvm;
 
 namespace llvm {
@@ -175,15 +175,17 @@ class CMKernelArgOffset : public ModulePass {
 
 char CMKernelArgOffset::ID = 0;
 
-INITIALIZE_PASS_BEGIN(CMKernelArgOffset, "cmkernelargoffset",
+INITIALIZE_PASS_BEGIN(CMKernelArgOffset, "CMKernelArgOffset",
                       "CM kernel arg offset determination", false, false)
-INITIALIZE_PASS_END(CMKernelArgOffset, "cmkernelargoffset",
+INITIALIZE_PASS_END(CMKernelArgOffset, "CMKernelArgOffset",
                     "CM kernel arg offset determination", false, false)
 
-Pass *llvm::createCMKernelArgOffsetPass(unsigned GrfByteSize,
-                                        bool UseBindlessImages) {
+namespace llvm {
+Pass *createCMKernelArgOffsetPass(unsigned GrfByteSize,
+                                  bool UseBindlessImages) {
   return new CMKernelArgOffset(GrfByteSize, UseBindlessImages);
 }
+} // namespace llvm
 
 // Check whether there is an input/output argument attribute.
 static bool canReorderArguments(const vc::KernelMetadata &KM) {

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXBIFFlagCtrlResolution.cpp

@@ -20,6 +20,7 @@ SPDX-License-Identifier: MIT
 #include <map>
 
 using namespace llvm;
+#define DEBUG_TYPE "GenXBIFFlagCtrlResolution"
 
 #define BIF_FLAG_CONTROL(BIF_FLAG_TYPE, BIF_FLAG_NAME)                            \
   BIF_FLAG_CTRL_N_S(BIF_FLAG_NAME),
@@ -145,7 +146,6 @@ bool GenXBIFFlagCtrlResolution::replace(T Value, GlobalVariable *GV) {
 
 char GenXBIFFlagCtrlResolution::ID = 0;
 
-
 INITIALIZE_PASS_BEGIN(GenXBIFFlagCtrlResolution, "GenXBIFFlagCtrlResolution",
                       "GenXBIFFlagCtrlResolution", false, false)
 

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXCloneIndirectFunctions.cpp

@@ -1,6 +1,6 @@
 /*========================== begin_copyright_notice ============================
 
-Copyright (C) 2022-2023 Intel Corporation
+Copyright (C) 2022-2024 Intel Corporation
 
 SPDX-License-Identifier: MIT
 
@@ -65,6 +65,7 @@ SPDX-License-Identifier: MIT
 #include <llvm/Transforms/Utils/Cloning.h>
 
 using namespace llvm;
+#define DEBUG_TYPE "GenXCloneIndirectFunctions"
 
 static cl::opt<bool> EnableCloneIndirectFunctions(
     "vc-enable-clone-indirect-functions",

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXPrintfResolution.cpp

@@ -54,6 +54,8 @@ SPDX-License-Identifier: MIT
 #include <sstream>
 #include <vector>
 
+#define DEBUG_TYPE "GenXPrintfResolution"
+
 using namespace llvm;
 using namespace vc;
 using namespace vc::bif::printf;

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXTranslateIntrinsics.cpp

@@ -25,7 +25,7 @@ SPDX-License-Identifier: MIT
 #include <llvm/Pass.h>
 #include <llvm/Support/Debug.h>
 
-#define DEBUG_TYPE "genx-translate-intrinsics"
+#define DEBUG_TYPE "GenXTranslateIntrinsics"
 
 using namespace llvm;
 

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXTranslateSPIRVBuiltins.cpp

@@ -36,7 +36,7 @@ SPDX-License-Identifier: MIT
 #include <llvm/Pass.h>
 #include <optional>
 
-#define DEBUG_TYPE "GENX_SPIRV_BUILTINS"
+#define DEBUG_TYPE "GenXTranslateSPIRVBuiltins"
 
 using namespace llvm;
 

IGC/VectorCompiler/lib/GenXOpts/CMTrans/GenXTypeLegalization.cpp

@@ -32,7 +32,7 @@ SPDX-License-Identifier: MIT
 #include <llvm/IR/InstVisitor.h>
 #include <llvm/Pass.h>
 
-#define DEBUG_TYPE "GENX_TYPE_LEGALIZATION"
+#define DEBUG_TYPE "GenXTypeLegalization"
 
 using namespace llvm;