[Technical debt] Remove compiler warnings for unused variable

niwinanto · niwinanto · commit e074eb265e89 · 2024-07-23T15:06:18.000+02:00
diff --git a/llvm/lib/Target/AIE/AIELiveRegs.cpp b/llvm/lib/Target/AIE/AIELiveRegs.cpp
@@ -53,20 +53,17 @@ void LiveRegs::computeBlockLiveIns(const MachineBasicBlock *MBB,
 }
 
 bool LiveRegs::equal(LivePhysRegs &CurrentLive, LivePhysRegs &OldLive) {
-  int LiveInCount = 0;
-  bool Equal = true;
-
+  auto CurrentLiveIt = CurrentLive.begin();
   for (const MCPhysReg Reg : OldLive) {
     if (!CurrentLive.contains(Reg)) {
-      Equal = false;
+      return false;
     }
-    LiveInCount++;
+    if (CurrentLiveIt == CurrentLive.end())
+      return false;
+    ++CurrentLiveIt;
   }
 
-  for (const MCPhysReg Reg : CurrentLive) {
-    LiveInCount--;
-  }
-  return LiveInCount == 0 && Equal;
+  return CurrentLiveIt == CurrentLive.end() ? true : false;
 }
 
 void LiveRegs::updateLiveRegs(LivePhysRegs &CurrentLive,

Original file line number	Diff line number	Diff line change
`@@ -53,20 +53,17 @@ void LiveRegs::computeBlockLiveIns(const MachineBasicBlock *MBB,`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`bool LiveRegs::equal(LivePhysRegs &CurrentLive, LivePhysRegs &OldLive) {`
`56`		`- int LiveInCount = 0;`
`57`		`- bool Equal = true;`
`58`		`-`
	`56`	`+ auto CurrentLiveIt = CurrentLive.begin();`
`59`	`57`	`for (const MCPhysReg Reg : OldLive) {`
`60`	`58`	`if (!CurrentLive.contains(Reg)) {`
`61`		`- Equal = false;`
	`59`	`+ return false;`
`62`	`60`	`}`
`63`		`- LiveInCount++;`
	`61`	`+ if (CurrentLiveIt == CurrentLive.end())`
	`62`	`+ return false;`
	`63`	`+ ++CurrentLiveIt;`
`64`	`64`	`}`
`65`	`65`
`66`		`- for (const MCPhysReg Reg : CurrentLive) {`
`67`		`- LiveInCount--;`
`68`		`- }`
`69`		`- return LiveInCount == 0 && Equal;`
	`66`	`+ return CurrentLiveIt == CurrentLive.end() ? true : false;`
`70`	`67`	`}`
`71`	`68`
`72`	`69`	`void LiveRegs::updateLiveRegs(LivePhysRegs &CurrentLive,`