[llvm-commits] CVS: llvm/lib/CodeGen/PrologEpilogInserter.cpp

[Chris Lattner]

Changes in directory llvm/lib/CodeGen:

PrologEpilogInserter.cpp updated: 1.40 -> 1.41
---
Log message:

Simplify/speedup the PEI by not having to scan for uses of the callee saved
registers.  This information is computed directly by the register allocator
now.


---
Diffs of the changes:  (+5 -19)

 PrologEpilogInserter.cpp |   24 +++++-------------------
 1 files changed, 5 insertions(+), 19 deletions(-)


Index: llvm/lib/CodeGen/PrologEpilogInserter.cpp
diff -u llvm/lib/CodeGen/PrologEpilogInserter.cpp:1.40 llvm/lib/CodeGen/PrologEpilogInserter.cpp:1.41
--- llvm/lib/CodeGen/PrologEpilogInserter.cpp:1.40	Sun Jan 23 15:45:01 2005
+++ llvm/lib/CodeGen/PrologEpilogInserter.cpp	Sun Jan 23 17:13:12 2005
@@ -111,8 +111,6 @@
       FrameSetupOpcode == -1 && FrameDestroyOpcode == -1)
     return;
 
-  // This bitset contains an entry for each physical register for the target...
-  std::vector<char> ModifiedRegs(RegInfo->getNumRegs());
   unsigned MaxCallFrameSize = 0;
   bool HasCalls = false;
 
@@ -127,19 +125,6 @@
         HasCalls = true;
         RegInfo->eliminateCallFramePseudoInstr(Fn, *BB, I++);
       } else {
-        for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) {
-          MachineOperand &MO = I->getOperand(i);
-          if (MO.isRegister() && MO.isDef()) {
-            assert(MRegisterInfo::isPhysicalRegister(MO.getReg()) &&
-                   "Register allocation must be performed!");
-            ModifiedRegs[MO.getReg()] = true;         // Register is modified
-          }
-
-          // Mark any implicitly defined registers as being modified.
-          for (const unsigned *ImpDefs = TII.getImplicitDefs(I->getOpcode());
-               *ImpDefs; ++ImpDefs)
-            ModifiedRegs[*ImpDefs] = true;
-        }
         ++I;
       }
 
@@ -150,14 +135,15 @@
   // Now figure out which *callee saved* registers are modified by the current
   // function, thus needing to be saved and restored in the prolog/epilog.
   //
+  const bool *PhysRegsUsed = Fn.getUsedPhysregs();
   for (unsigned i = 0; CSRegs[i]; ++i) {
     unsigned Reg = CSRegs[i];
-    if (ModifiedRegs[Reg]) {
-      RegsToSave.push_back(Reg);  // If modified register...
+    if (PhysRegsUsed[Reg]) {
+      RegsToSave.push_back(Reg);  // If the reg is modified, save it!
     } else {
       for (const unsigned *AliasSet = RegInfo->getAliasSet(Reg);
-           *AliasSet; ++AliasSet) {  // Check alias registers too...
-        if (ModifiedRegs[*AliasSet]) {
+           *AliasSet; ++AliasSet) {  // Check alias registers too.
+        if (PhysRegsUsed[*AliasSet]) {
           RegsToSave.push_back(Reg);
           break;
         }