WIP ldc-developers#539 resetAll made up several percent running time …

…for phobos
Trass3r · Jun 30, 2014 · bbb2bb8 · bbb2bb8
1 parent 407adff
commit bbb2bb8
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 29 deletions.
diff --git a/ir/irdsymbol.cpp b/ir/irdsymbol.cpp
@@ -12,55 +12,43 @@
 #include "ir/irdsymbol.h"
 #include "ir/irvar.h"
 
-std::set<IrDsymbol*> IrDsymbol::list;
+std::vector<IrDsymbol*> IrDsymbol::list;
 
 void IrDsymbol::resetAll()
 {
     Logger::println("resetting %zu Dsymbols", list.size());
-    std::set<IrDsymbol*>::iterator it;
-    for(it = list.begin(); it != list.end(); ++it)
+
+    for (std::vector<IrDsymbol*>::iterator it = list.begin(), end = list.end(); it != end; ++it)
         (*it)->reset();
 }
 
 IrDsymbol::IrDsymbol()
 {
-    bool incr = list.insert(this).second;
-    assert(incr);
+    list.push_back(this);
     reset();
 }
 
 IrDsymbol::IrDsymbol(const IrDsymbol& s)
 {
-    bool incr = list.insert(this).second;
-    assert(incr);
-    DModule = s.DModule;
-    irModule = s.irModule;
-    irAggr = s.irAggr;
-    irFunc = s.irFunc;
-    resolved = s.resolved;
-    declared = s.declared;
-    initialized = s.initialized;
-    defined = s.defined;
-    irGlobal = s.irGlobal;
-    irLocal = s.irLocal;
-    irField = s.irField;
+    list.push_back(this);
+    memcpy(this, &s, sizeof(IrDsymbol));
 }
 
 IrDsymbol::~IrDsymbol()
 {
-    list.erase(this);
+    if (this == list.back())
+    {
+        list.pop_back();
+        return;
+    }
+
+    std::vector<IrDsymbol*>::iterator it = std::find(list.rbegin(), list.rend(), this).base();
+    list.erase(it);
 }
 
 void IrDsymbol::reset()
 {
-    DModule = NULL;
-    irModule = NULL;
-    irAggr = NULL;
-    irFunc = NULL;
-    resolved = declared = initialized = defined = false;
-    irGlobal = NULL;
-    irLocal = NULL;
-    irField = NULL;
+    memset(this, 0, sizeof(IrDsymbol));
 }
 
 bool IrDsymbol::isSet()

diff --git a/ir/irdsymbol.h b/ir/irdsymbol.h
@@ -14,7 +14,7 @@
 #ifndef LDC_IR_IRDSYMBOL_H
 #define LDC_IR_IRDSYMBOL_H
 
-#include <set>
+#include <vector>
 
 struct IrModule;
 struct IrFunction;
@@ -33,7 +33,7 @@ namespace llvm {
 
 struct IrDsymbol
 {
-    static std::set<IrDsymbol*> list;
+    static std::vector<IrDsymbol*> list;
     static void resetAll();
 
     // overload all of these to make sure