Check whether G4_RegVar is GRF when marking busy in local RA

pratikashar · igcbot · commit c8a7ccf59153 · 2024-07-30T17:40:29.000+02:00
Local RA allocates GRF variables only. When a variable is marked as
live-in and live-out, we need to mark it in appropriate local RA data
structures so those GRFs are never used for assignment.

Check whether G4_RegVar is GRF when marking busy in local RA.
diff --git a/visa/LocalRA.cpp b/visa/LocalRA.cpp
@@ -2104,7 +2104,7 @@ void PhyRegsLocalRA::markPhyRegs(G4_Declare *topdcl) {
   unsigned numwords = 0;
   unsigned int regnum = 0;
 
-  if (rvar->getPhyReg() == nullptr) {
+  if (!rvar->getPhyReg() || !rvar->isGreg()) {
     return;
   }
   // Calculate number of physical registers required by this dcl

Original file line number	Diff line number	Diff line change
`@@ -2104,7 +2104,7 @@ void PhyRegsLocalRA::markPhyRegs(G4_Declare *topdcl) {`
`2104`	`2104`	`unsigned numwords = 0;`
`2105`	`2105`	`unsigned int regnum = 0;`
`2106`	`2106`
`2107`		`- if (rvar->getPhyReg() == nullptr) {`
	`2107`	`+ if (!rvar->getPhyReg() \|\| !rvar->isGreg()) {`
`2108`	`2108`	`return;`
`2109`	`2109`	`}`
`2110`	`2110`	`// Calculate number of physical registers required by this dcl`