[type profile] Collect return types.

src/compiler/bytecode-graph-builder.h

Index: src/compiler/bytecode-graph-builder.h
diff --git a/src/compiler/bytecode-graph-builder.h b/src/compiler/bytecode-graph-builder.h
index 3e73e030709dc4b4f49eaa1545858a9afa26e328..6178c180925304f984f0388532f04934b3ca9902 100644
--- a/src/compiler/bytecode-graph-builder.h
+++ b/src/compiler/bytecode-graph-builder.h
@@ -84,6 +84,12 @@ class BytecodeGraphBuilder {
     return MakeNode(op, arraysize(buffer), buffer, false);
   }
 
+  Node* NewNode(const Operator* op, Node* n1, Node* n2, Node* n3, Node* n4,

[ahaas, 2017/03/20 09:34:43]

You could be super fancy and do this with variadic templates. Instead of
introducing a new NewNode function, you could replace all NewNode functions by
  template<typename... Nodes>
  Node* NewNode(const Operator* op, Nodes... n) {
    Node* buffer[] = {n...};
    return MakeNode(op, arraysize(buffer), buffer, false);
  }

[ahaas, 2017/03/20 09:53:36]

better use

  template<typename... Nodes>
  Node* NewNode(const Operator* op, Node* n0, Nodes... n) {
    Node* buffer[] = {n0, n...};
    return MakeNode(op, arraysize(buffer), buffer, false);
  }

because otherwise there could be a signature overlap with 
  Node* NewNode(const Operator* op, bool incomplete = false)

[Franzi, 2017/03/20 15:38:39]

No chance to be super fancy :( Don't need NewNode() with 5 nodes anymore. 

+                Node* n5) {
+    Node* buffer[] = {n1, n2, n3, n4, n5};
+    return MakeNode(op, arraysize(buffer), buffer, false);
+  }
+
   // Helpers to create new control nodes.
   Node* NewIfTrue() { return NewNode(common()->IfTrue()); }
   Node* NewIfFalse() { return NewNode(common()->IfFalse()); }