IGA: fix a bug in swsb setter to remove the assert for checking DepSet companion

Change-Id: I907290b64f06aa9cbe7e876f2c1f5d62ddfe20e3

Author: DianaChen

visa/iga/GEDLibrary/GED_external/Source/common/version.cpp

@@ -24,4 +24,4 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 ======================= end_copyright_notice ==================================*/
 
-const char* gedVersion = "0.36 (cd0b9a7e)";
+const char* gedVersion = "0.37 (8a8db73d)";

visa/iga/IGALibrary/Frontend/KernelParser.cpp

@@ -1906,8 +1906,15 @@ class KernelParser : GenParser
         if (dty != Type::INVALID) {
             int typeSize = TypeSizeInBits(dty)/8;
             if (!ri.isSubRegByteOffsetValid(regNum, subregNum * typeSize, m_model.getGRFByteSize())) {
-                Error(subregLoc,
-                    "subregister out of bounds for data type", ToSyntax(dty));
+                if (ri.regName == RegName::GRF_R) {
+                    Error(subregLoc,
+                        "subregister out of bounds for data type", ToSyntax(dty));
+                } else {
+                    // Print warning for non-GRF register, in case for those valid but strange case
+                    // e.g. null0.20:f
+                    Warning(subregLoc,
+                        "subregister out of bounds for data type");
+                }
             } else if (typeSize < ri.accGran) {
                 Warning(regnameLoc, "access granularity too small for data type");
             }

visa/iga/IGALibrary/IR/SWSBSetter.cpp

@@ -335,7 +335,7 @@ void SWSBAnalyzer::calculateDependence(DepSet &currDep, SWSB &distanceDependency
                         }
                         // clear this instruction's dependency since it is satisfied
                         clearDepBuckets(*dep);
-                        assert(dep->getCompanion() != nullptr);
+
                         // clear its companion because when an in-order instruction is synced, both its
                         // input and output dependency are satisfied. The only case is that if it has
                         // read/write_always_interfere dependency, it should be reserved.
@@ -346,9 +346,13 @@ void SWSBAnalyzer::calculateDependence(DepSet &currDep, SWSB &distanceDependency
                         //      mov (1|M0)               r104.0<1>:ud  sr0.1<0;1,0>:ud
                         //      cmp(16 | M0)   (ne)f0.0   null:ud    r104.0<0; 1, 0> : ub   r62.4<0; 1, 0> : uw
                         // A@1 is required for cmp instead of I@1
-                        if (dep->getCompanion()->getDepType() != DEP_TYPE::WRITE_ALWAYS_INTERFERE &&
-                            dep->getCompanion()->getDepType() != DEP_TYPE::READ_ALWAYS_INTERFERE)
-                            clearDepBuckets(*dep->getCompanion());
+                        if (dep->getCompanion() != nullptr) {
+                            // In the case that this DepSet is generated from math_wa_info, it won't have companion
+                            if (dep->getCompanion()->getDepType() != DEP_TYPE::WRITE_ALWAYS_INTERFERE &&
+                                dep->getCompanion()->getDepType() != DEP_TYPE::READ_ALWAYS_INTERFERE) {
+                                clearDepBuckets(*dep->getCompanion());
+                            }
+                        }
                     } // end of if (prevDepClass == DEP_CLASS::IN_ORDER)
                     else // prev is out of order
                     {

visa/iga/IGALibrary/Models/Models.cpp

@@ -75,8 +75,7 @@ static const struct RegInfo REGISTER_SPECIFICATIONS[] = {
         0x2, 0,
         1,
         2, (32,32)),
-    IGA_REGISTER_SPEC_GE(
-        Platform::GEN12P1,
+    IGA_REGISTER_SPEC(Platform::GEN12P1, Platform::GEN12P1,
         RegName::ARF_ACC, "acc", "Accumulator",
         0x2, 0,
         1,
@@ -87,14 +86,11 @@ static const struct RegInfo REGISTER_SPECIFICATIONS[] = {
         0x2, 2, // offset by 2 "acc2-9"
         4,
         8, (32,32,32,32,32,32,32,32)),
-    IGA_REGISTER_SPEC_GE(
-        Platform::GEN12P1,
+    IGA_REGISTER_SPEC(Platform::GEN12P1, Platform::GEN12P1,
         RegName::ARF_MME, "mme", "Math Macro",
         0x2, 8, // offset by 8 "acc8-15"
         4,
         8, (32,32,32,32,32,32,32,32)),
-
-
     IGA_REGISTER_SPEC_UNIFORM(
         RegName::ARF_F, "f", "Flag Register",
         0x3, 0,

visa/iga/IGALibrary/version.hpp

@@ -26,6 +26,6 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 #ifndef _IGA_VERSION_HPP_
 #define _IGA_VERSION_HPP_
-#define IGA_VERSION_STRING "0.15.12"
+#define IGA_VERSION_STRING "0.15.13"
 
 #endif // _IGA_VERSION_HPP_