[compiler] Collect eager inner functions for compilation during renumbering.

src/zone/zone-containers.h

Index: src/zone/zone-containers.h
diff --git a/src/zone/zone-containers.h b/src/zone/zone-containers.h
index 7a53d4b25950c2fce7613e5cd22ab534cf0b8fe8..d59fb3a74368d924dd9cd2ea95c3033b3640e05d 100644
--- a/src/zone/zone-containers.h
+++ b/src/zone/zone-containers.h
@@ -13,6 +13,7 @@
 #include <stack>
 #include <vector>
 
+#include "src/utils.h"

[marja, 2017/01/10 10:10:43]

Why?

[rmcilroy, 2017/01/11 10:20:11]

You are right , this wasn't needed (removed anyway since I moved the helper into
utils.h instead).

 #include "src/zone/zone-allocator.h"
 
 namespace v8 {
@@ -139,6 +140,20 @@ class ZoneMultimap
 typedef ZoneVector<bool> BoolVector;
 typedef ZoneVector<int> IntVector;
 
+// Can be used to create a threaded list of |T|.
+template <typename T>
+class ThreadedListZoneEntry final : public ZoneObject {

[Michael Starzinger, 2017/01/10 12:26:09]

This file is specifically meant to contain stdlib containers living on the zone
(and specifically don't subclass ZoneObject). Could we move this helper class
into the utils.h file instead, close to our hand-woven ThreadedList class?

[rmcilroy, 2017/01/11 10:20:11]

I had it there originally but it means src/utils.h needs to to include
src/zone/zone.h (for ZoneObject), however that created a circular dependency
since zone.h already includes src/utils.h (indirectly through src/list.h). 

It turns out list.h only needed vector.h, so I broke the cycle  and moved this
back into utils.h, however it needed a bit of include munging across the
codebase.

+ public:
+  explicit ThreadedListZoneEntry(T value) : value_(value), next_(nullptr) {}
+
+  T value() { return value_; }
+  ThreadedListZoneEntry<T>** next() { return &next_; }
+
+ private:
+  T value_;
+  ThreadedListZoneEntry<T>* next_;
+};

[jochen (gone - plz use gerrit), 2017/01/10 09:57:39]

disallow copy/assign?

[rmcilroy, 2017/01/11 10:20:11]

Done.

+
 }  // namespace internal
 }  // namespace v8