Update verifySavedState()'s comment, so that it reflects its current

status.  In doFinalization(), skip over external functions, just like
Anand's mapping info does.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@9703 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/CodeGen/RegAlloc/PhyRegAlloc.cpp b/lib/CodeGen/RegAlloc/PhyRegAlloc.cpp
index 65d1b70e3e018d004fc1761f135ba453f91f8627..42150232d2211503a578b2447b09665d9776f070 100644 (file)
--- a/lib/CodeGen/RegAlloc/PhyRegAlloc.cpp
+++ b/lib/CodeGen/RegAlloc/PhyRegAlloc.cpp
@@ -1177,7 +1177,8 @@ void PhyRegAlloc::saveState () {
 
 
 /// Check the saved state filled in by saveState(), and abort if it looks
-/// wrong. Only used when debugging.
+/// wrong. Only used when debugging. FIXME: Currently it just prints out
+/// the state, which isn't quite as useful.
 ///
 void PhyRegAlloc::verifySavedState () {
   std::vector<AllocInfo> &state = FnAllocState[Fn];
@@ -1233,6 +1234,7 @@ bool PhyRegAlloc::doFinalization (Module &M) {
   std::vector<Constant *> allstate;
   for (Module::iterator I = M.begin (), E = M.end (); I != E; ++I) {
     Function *F = I;
+    if (F->isExternal ()) continue;
     if (FnAllocState.find (F) == FnAllocState.end ()) {
       allstate.push_back (ConstantPointerNull::get (PT));
     } else {

diff --git a/lib/Target/SparcV9/RegAlloc/PhyRegAlloc.cpp b/lib/Target/SparcV9/RegAlloc/PhyRegAlloc.cpp
index 65d1b70e3e018d004fc1761f135ba453f91f8627..42150232d2211503a578b2447b09665d9776f070 100644 (file)
--- a/lib/Target/SparcV9/RegAlloc/PhyRegAlloc.cpp
+++ b/lib/Target/SparcV9/RegAlloc/PhyRegAlloc.cpp
@@ -1177,7 +1177,8 @@ void PhyRegAlloc::saveState () {
 
 
 /// Check the saved state filled in by saveState(), and abort if it looks
-/// wrong. Only used when debugging.
+/// wrong. Only used when debugging. FIXME: Currently it just prints out
+/// the state, which isn't quite as useful.
 ///
 void PhyRegAlloc::verifySavedState () {
   std::vector<AllocInfo> &state = FnAllocState[Fn];
@@ -1233,6 +1234,7 @@ bool PhyRegAlloc::doFinalization (Module &M) {
   std::vector<Constant *> allstate;
   for (Module::iterator I = M.begin (), E = M.end (); I != E; ++I) {
     Function *F = I;
+    if (F->isExternal ()) continue;
     if (FnAllocState.find (F) == FnAllocState.end ()) {
       allstate.push_back (ConstantPointerNull::get (PT));
     } else {