Remove the WAR predicate dependence

HW can cover this kind dependence

Author: bcheng0127

visa/HWCaps.inc

@@ -926,6 +926,10 @@ bool needA0WAR() const {
   return (getPlatform() >= Xe2);
 }
 
+bool needPredicateWAR() const {
+  return (getPlatform() < Xe3);
+}
+
 bool alwaysAllowGlobalFlagOpt() const {
   // We shouldn't add this kind of ugly platform check. However, there's
   // a complicated bug found in TGLLP when disabling a flagopt case. There

visa/LocalScheduler/SWSB_G4IR.cpp

@@ -6101,8 +6101,12 @@ bool G4_BB_SB::getGRFFootPrint(SBNode *node, PointsToAnalysis &p) {
   for (G4_INST *inst : node->instVec) {
     hasDistOneAReg |=
         getGRFFootPrintOperands(node, inst, Opnd_src0, Opnd_src4, p);
-    hasDistOneAReg |=
-        getGRFFootPrintOperands(node, inst, Opnd_pred, Opnd_implAccDst, p);
+    if (builder.needPredicateWAR())
+      hasDistOneAReg |=
+          getGRFFootPrintOperands(node, inst, Opnd_pred, Opnd_implAccDst, p);
+    else
+      hasDistOneAReg |=
+          getGRFFootPrintOperands(node, inst, Opnd_condMod, Opnd_implAccDst, p);
     hasDistOneAReg |=
         getGRFFootPrintOperands(node, inst, Opnd_dst, Opnd_dst, p);
   }
@@ -6116,7 +6120,12 @@ void G4_BB_SB::getGRFBucketDescs(SBNode *node, std::vector<SBBucketDesc> &BDvec,
   // scan order is src0, src1, src2, src3, src4, dst
   getGRFBucketsForOperands(node, Opnd_src0, Opnd_src4, BDvec, GRFOnly);
   if (!GRFOnly) {
-    getGRFBucketsForOperands(node, Opnd_pred, Opnd_implAccDst, BDvec, GRFOnly);
+    if (builder.needPredicateWAR())
+      getGRFBucketsForOperands(node, Opnd_pred, Opnd_implAccDst, BDvec,
+                               GRFOnly);
+    else
+      getGRFBucketsForOperands(node, Opnd_condMod, Opnd_implAccDst, BDvec,
+                               GRFOnly);
   }
   getGRFBucketsForOperands(node, Opnd_dst, Opnd_dst, BDvec, GRFOnly);