[Autobackout][FunctionalRegression]Revert of change: 1084303: Remove WAR dependence for flag register

sys-igc · igcbot · commit edc7e6d4addf · 2025-11-25T20:20:30.000+01:00
Remove WAR dependence for flag register
diff --git a/visa/HWCaps.inc b/visa/HWCaps.inc
@@ -926,10 +926,6 @@ bool needA0WAR() const {
   return (getPlatform() >= Xe2);
 }
 
-bool needFlagWAR() 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
diff --git a/visa/LocalScheduler/SWSB_G4IR.cpp b/visa/LocalScheduler/SWSB_G4IR.cpp
@@ -5898,7 +5898,7 @@ bool G4_BB_SB::getFootprintForOperand(SBNode *node, G4_INST *inst,
       footprint = getFootprintForACC(opnd, opndNum, inst);
       node->setFootprint(footprint, opndNum);
     }
-    if (builder.needFlagWAR() && isFlagReg) {
+    if (isFlagReg) {
       footprint = getFootprintForFlag(opnd, opndNum, inst);
       node->setFootprint(footprint, opndNum);
     }

Original file line number	Diff line number	Diff line change
`@@ -5898,7 +5898,7 @@ bool G4_BB_SB::getFootprintForOperand(SBNode node, G4_INST inst,`
`5898`	`5898`	`footprint = getFootprintForACC(opnd, opndNum, inst);`
`5899`	`5899`	`node->setFootprint(footprint, opndNum);`
`5900`	`5900`	`}`
`5901`		`- if (builder.needFlagWAR() && isFlagReg) {`
	`5901`	`+ if (isFlagReg) {`
`5902`	`5902`	`footprint = getFootprintForFlag(opnd, opndNum, inst);`
`5903`	`5903`	`node->setFootprint(footprint, opndNum);`
`5904`	`5904`	`}`