IGCMetrics - fix issues with VISAOffset

Some of the instructions dosen't have VISAOffset - mostly, because those instructions are a dead code.
IGCMetrics during matching Spill/Fills was facing such scenerio - now, we are ignoring load/store without VISAOffest.

Author: lwesiers

IGC/DebugInfo/VISAModule.cpp

@@ -165,6 +165,10 @@ void VISAModule::BeginEncodingMark() { ValidateVisaId(); }
 
 void VISAModule::EndEncodingMark() { UpdateVisaId(); }
 
+bool VISAModule::HasVisaOffset(const llvm::Instruction *pInst) const {
+  return m_instInfoMap.find(pInst) != m_instInfoMap.end();
+}
+
 unsigned int VISAModule::GetVisaOffset(const llvm::Instruction *pInst) const {
   InstInfoMap::const_iterator itr = m_instInfoMap.find(pInst);
   IGC_ASSERT_MESSAGE(itr != m_instInfoMap.end(), "Invalid Instruction");

IGC/DebugInfo/VISAModule.hpp

@@ -448,6 +448,11 @@ class VISAModule {
   /// @brief Mark end of VISA code emitting section.
   void EndEncodingMark();
 
+  /// @brief Return true if instruction has VISA offset.
+  /// @param Instruction to query.
+  /// @return Return true if instruction has VISA offset, otherwise false.
+  bool HasVisaOffset(const llvm::Instruction *pInst) const;
+
   /// @brief Return VISA offset (in instructions) mapped to given instruction.
   /// @param Instruction to query.
   /// @return VISA offset (in instructions)

IGC/Metrics/IGCMetricImpl.cpp

@@ -1084,13 +1084,18 @@ namespace IGCMetrics
             auto spillMD = instr.getMetadata(metric->fillInstrKindID);
             if (spillMD != nullptr)
             {
-                auto func_m = metric->GetFuncMetric(&instr);
-                auto spillFill_m = func_m->mutable_spillfill_stats();
+                // Some instructions dosen't have visa offset
+                // that means the instruction is a dead code
+                if (CurrentVISA->HasVisaOffset(&instr))
+                {
+                    auto func_m = metric->GetFuncMetric(&instr);
+                    auto spillFill_m = func_m->mutable_spillfill_stats();
 
-                spillFill_m->set_countfillinstr(spillFill_m->countfillinstr() + 1);
+                    spillFill_m->set_countfillinstr(spillFill_m->countfillinstr() + 1);
 
-                // Add spill instruction to metrics
-                spillFill_m->add_fillinstrvisaid(CurrentVISA->GetVisaOffset(&instr));
+                    // Add spill instruction to metrics
+                    spillFill_m->add_fillinstrvisaid(CurrentVISA->GetVisaOffset(&instr));
+                }
             }
         }
 
@@ -1099,13 +1104,18 @@ namespace IGCMetrics
             auto fillMD = instr.getMetadata(metric->spillInstrKindID);
             if (fillMD != nullptr)
             {
-                auto func_m = metric->GetFuncMetric(&instr);
-                auto spillFill_m = func_m->mutable_spillfill_stats();
+                // Some instructions dosen't have visa offset
+                // that means the instruction is a dead code
+                if (CurrentVISA->HasVisaOffset(&instr))
+                {
+                    auto func_m = metric->GetFuncMetric(&instr);
+                    auto spillFill_m = func_m->mutable_spillfill_stats();
 
-                spillFill_m->set_countspillinstr(spillFill_m->countspillinstr() + 1);
+                    spillFill_m->set_countspillinstr(spillFill_m->countspillinstr() + 1);
 
-                // Add spill instruction to metrics
-                spillFill_m->add_spillinstrvisaid(CurrentVISA->GetVisaOffset(&instr));
+                    // Add spill instruction to metrics
+                    spillFill_m->add_spillinstrvisaid(CurrentVISA->GetVisaOffset(&instr));
+                }
             }
         }
     };