Subzero: Rewrite the pass timing infrastructure.

src/IceOperand.cpp

Index: src/IceOperand.cpp
diff --git a/src/IceOperand.cpp b/src/IceOperand.cpp
index d1854f4f0f0a6d4e48306b87acef22f227f2b84b..3a10a6a5159470ad5be882aa75e586d72706bed3 100644
--- a/src/IceOperand.cpp
+++ b/src/IceOperand.cpp
@@ -122,9 +122,19 @@ bool LiveRange::overlaps(const LiveRange &Other) const {
 }
 
 bool LiveRange::overlaps(InstNumberT OtherBegin) const {

[Jim Stichnoth, 2014/09/28 14:26:39]

This is called quite a bit, so provide a better implementation.

(Tested by computing both ways and asserting the same result.)

+  for (RangeType::const_iterator I = Range.begin(), E = Range.end(); I != E;
+       ++I) {
+    if (OtherBegin >= I->second)
+      return false;
+    if (I->first <= OtherBegin)
+      return true;
+  }
+  return false;
+#if 0 // An equivalent but less inefficient implementation:
   LiveRange Temp;
   Temp.addSegment(OtherBegin, OtherBegin + 1);
   return overlaps(Temp);
+#endif
 }
 
 // Returns true if the live range contains the given instruction
@@ -259,6 +269,7 @@ const Inst *VariableTracking::getSingleDefinition() const {
 }
 
 void VariablesMetadata::init() {
+  TimerMarker T("vmetadata", Func->getContext());
   Metadata.clear();
   Metadata.resize(Func->getNumVariables());