Const-correct == operator in LocalScheduler

Const-correct == operator in LocalScheduler

Author: jaladreips

IGC/Compiler/CISACodeGen/GenCodeGenModule.h

@@ -104,11 +104,11 @@ namespace IGC {
             {
                 return *minor;
             }
-            bool operator==(const iterator& rhs)
+            bool operator==(const iterator& rhs) const
             {
                 return (major == rhs.major && minor == rhs.minor);
             }
-            bool operator!=(const iterator& rhs)
+            bool operator!=(const iterator& rhs) const
             {
                 return !(*this == rhs);
             }

IGC/Compiler/Optimizer/OpenCLPasses/KernelArgs/KernelArgs.cpp

@@ -1029,15 +1029,15 @@ const KernelArg& KernelArgs::const_iterator::operator*()
     return *m_minor;
 }
 
-bool KernelArgs::const_iterator::operator!=(const const_iterator& iterator)
+bool KernelArgs::const_iterator::operator!=(const const_iterator& iterator) const
 {
     if (m_empty)
         return (m_major != iterator.m_major);
     else
         return (m_major != iterator.m_major) || (m_minor != iterator.m_minor);
 }
 
-bool KernelArgs::const_iterator::operator==(const const_iterator& iterator)
+bool KernelArgs::const_iterator::operator==(const const_iterator& iterator) const
 {
     if (m_empty)
         return (m_major == iterator.m_major);

IGC/Compiler/Optimizer/OpenCLPasses/KernelArgs/KernelArgs.hpp

@@ -459,13 +459,13 @@ namespace IGC
             ///         by checking if they point to the same element
             /// @param  iterator    An iterator to compare this iterator
             /// @return true if the iterators pare different, false otherwise
-            bool operator!=(const const_iterator& iterator);
+            bool operator!=(const const_iterator& iterator) const;
 
             /// @brief  Checks whether this iterator and the given iterator are same
             ///         by checking if they point to the same element
             /// @param  iterator    An iterator to compare this iterator
             /// @return true if the iterators are same, false otherwise
-            bool operator==(const const_iterator& iterator);
+            bool operator==(const const_iterator& iterator) const;
 
         private:
             AllocationArgs::const_iterator          m_major;

IGC/Compiler/Optimizer/PreCompiledFuncImport.hpp

@@ -204,7 +204,7 @@ namespace IGC
 
             void updateUses();
 
-            bool operator==(const llvm::Function* F) {
+            bool operator==(const llvm::Function* F) const {
                 return this->F == F;
             }
 

visa/LocalScheduler/LocalScheduler_G4IR.cpp

@@ -653,13 +653,13 @@ class LiveBuckets {
              node_it != LB->nodeBucketsArray[bucket].bucketVec.end());
       return *this;
     }
-    bool operator==(const BN_iterator &it2) {
+    bool operator==(const BN_iterator &it2) const {
       vASSERT(LB == it2.LB && iterateAll == it2.iterateAll);
       // NOTE: order of comparisons matters: if different buckets
       //       then node_its are of different vectors
       return (bucket == it2.bucket && node_it == it2.node_it);
     }
-    bool operator!=(const BN_iterator &it2) {
+    bool operator!=(const BN_iterator &it2) const {
       vASSERT(LB == it2.LB && iterateAll == it2.iterateAll);
       return (!(*this == it2));
     }