Internal refactor

Change-Id: Ie8eaee983c7b3d8b175d905061d1996723308f64

Author: unboagable

IGC/Compiler/CISACodeGen/CISABuilder.cpp

@@ -2508,7 +2508,7 @@ void CEncoder::SetFloatDenormMode(VISAKernel* vKernel, Float_DenormMode mode16,
     VISA_VectorOpnd* src0_Opnd = nullptr;
     VISA_VectorOpnd* src1_Opnd = nullptr;
     VISA_VectorOpnd* dst_Opnd = nullptr;
-    VISA_GenVar* cr0_var;
+    VISA_GenVar* cr0_var = nullptr;
     uint imm_data = 0;
     if (mode16 == FLOAT_DENORM_RETAIN)
         imm_data |= 0x400;

IGC/Compiler/CISACodeGen/CodeSinking.cpp

@@ -759,8 +759,8 @@ bool CodeSinking::checkCongruent(const InstPair& values,
             return false;
         }
         unsigned as = ld0->getPointerAddressSpace();
-        unsigned bufId;
-        bool directBuf;
+        unsigned bufId = 0;
+        bool directBuf = false;
         BufferType bufType = IGC::DecodeAS4GFXResource(as, directBuf, bufId);
         if (bufType != CONSTANT_BUFFER)
         {

IGC/Compiler/CISACodeGen/PatternMatchPass.cpp

@@ -503,7 +503,8 @@ CodeGenPatternMatch::isFPToIntegerSatWithExactConstant(llvm::CastInst *I) {
             return std::make_tuple(nullptr, 0, ISA_TYPE_F);
     }
 
-    llvm::ConstantFP *CMin, *CMax;
+    llvm::ConstantFP *CMin = nullptr;
+    llvm::ConstantFP *CMax = nullptr;
     llvm::Value *X = nullptr;
 
     if (!match(I->getOperand(0), m_ClampWithConstants(m_Value(X), CMin, CMax)))
@@ -627,7 +628,8 @@ CodeGenPatternMatch::isFPToUnsignedIntSatWithInexactConstant( llvm::SelectInst *
 
     // Fold extra clamp.
     Value *X2 = nullptr;
-    ConstantFP *CMin2, *CMax2;
+    ConstantFP *CMin2 = nullptr;
+    ConstantFP *CMax2 = nullptr;
     if (match(X, m_ClampWithConstants(m_Value(X2), CMin2, CMax2))) {
         if (CMin2 == FMin) {
             if (CMax2->isExactlyValue(255.0)) {
@@ -2529,7 +2531,7 @@ bool CodeGenPatternMatch::MatchSatModifier(llvm::Instruction& I)
         }
     };
     bool match = false;    
-    llvm::Value* source;
+    llvm::Value* source = nullptr;
     if(isSat(&I, source))
     {
         SatPattern *satPattern = new (m_allocator) SatPattern();

IGC/Compiler/CISACodeGen/VectorPreProcess.cpp

@@ -583,7 +583,7 @@ bool VectorPreProcess::splitStore(AbstractStoreInst& ASI, V2SMap& vecToSubVec)
     if (svals.size() == 0)
     {
         // Need to create splitted values.
-        Instruction *insertBeforeInst;
+        Instruction *insertBeforeInst = nullptr;
         Value* scalars[VP_MAX_VECTOR_SIZE];
         getOrGenScalarValues(*SI->getParent()->getParent(),
                              StoredVal, scalars, insertBeforeInst);

IGC/Compiler/CustomUnsafeOptPass.cpp

@@ -1344,11 +1344,11 @@ bool CustomUnsafeOptPass::visitExchangeCB(llvm::BinaryOperator &I)
         return false;
     }
 
-    unsigned bufId;
+    unsigned bufId = 0;
     unsigned cbIndex0 = 0;
     unsigned cbIndex1 = 0;
     unsigned hasCB = 0;
-    bool directBuf;
+    bool directBuf = false;
 
     for (int i = 0; i < 2; i++)
     {
@@ -2680,7 +2680,12 @@ bool EarlyOutPatterns::DotProductMatch(const Instruction *I)
 
     using namespace PatternMatch;
 
-	Value *X1, *Y1, *Z1, *X2, *Y2, *Z2;
+    Value *X1 = nullptr;
+    Value *Y1 = nullptr;
+    Value *Z1 = nullptr;
+    Value *X2 = nullptr;
+    Value *Y2 = nullptr;
+    Value *Z2 = nullptr;
 
 	// dp3
 

IGC/Compiler/PromoteResourceToDirectAS.cpp

@@ -180,7 +180,7 @@ void PromoteResourceToDirectAS::PromoteSamplerTextureToDirectAS(GenIntrinsicInst
         // Only try to promote bindless pointers ( as(1) or as(2) ), or indirect buffer access
         return;
     }
-    unsigned bufID;
+    unsigned bufID = 0;
     BufferType bufTy;
     BufferAccessType accTy;
     bool canPromote = false;