Change implementation of LiveIntervals::overlap(). This results in a

30-50% decrease in running time of the linear scan register allocator.

llvm-svn: 10707

1 files changed, 22 insertions, 10 deletions

diff --git a/llvm/lib/CodeGen/LiveIntervals.cpp b/llvm/lib/CodeGen/LiveIntervals.cpp
index 1bd8deb37b2..fe47a3661d8 100644
--- a/llvm/lib/CodeGen/LiveIntervals.cpp
+++ b/llvm/lib/CodeGen/LiveIntervals.cpp
@@ -386,17 +386,29 @@ bool LiveIntervals::Interval::liveAt(unsigned index) const
 
 bool LiveIntervals::Interval::overlaps(const Interval& other) const
 {
-    std::vector<bool> bitMap(end(), false);
-    for (Ranges::const_iterator r = ranges.begin(); r != ranges.end(); ++r) {
-        for (unsigned i = r->first; i < r->second; ++i)
-            bitMap[i] = true;
-    }
-    for (Ranges::const_iterator r = other.ranges.begin();
-         r != other.ranges.end(); ++r) {
-        for (unsigned i = r->first;
-             i < r->second && i < bitMap.size(); ++i)
-            if (bitMap[i])
+    Ranges::const_iterator i = ranges.begin();
+    Ranges::const_iterator j = other.ranges.begin();
+
+    while (i != ranges.end() && j != other.ranges.end()) {
+        if (i->first < j->first) {
+            if (i->second > j->first) {
+                return true;
+            }
+            else {
+                ++i;
+            }
+        }
+        else if (j->first < i->first) {
+            if (j->second > i->first) {
                 return true;
+            }
+            else {
+                ++j;
+            }
+        }
+        else {
+            return true;
+        }
     }
 
     return false;