Load runtime entries from the Thread instead of the ObjectPool.

runtime/vm/runtime_entry_list.h

Index: runtime/vm/runtime_entry_list.h
diff --git a/runtime/vm/runtime_entry_list.h b/runtime/vm/runtime_entry_list.h
new file mode 100644
index 0000000000000000000000000000000000000000..e24dd54f082b66595dd82088b22a27d8a9ad01dc
--- /dev/null
+++ b/runtime/vm/runtime_entry_list.h
@@ -0,0 +1,68 @@
+// Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+#ifndef VM_RUNTIME_ENTRY_LIST_H_
+#define VM_RUNTIME_ENTRY_LIST_H_
+
+namespace dart {
+
+#define RUNTIME_ENTRY_LIST(V)                                                  \
+  V(AllocateArray)                                                             \
+  V(AllocateContext)                                                           \
+  V(AllocateObject)                                                            \
+  V(BreakpointRuntimeHandler)                                                  \
+  V(SingleStepHandler)                                                         \
+  V(CloneContext)                                                              \
+  V(FixCallersTarget)                                                          \
+  V(FixAllocationStubTarget)                                                   \
+  V(InlineCacheMissHandlerOneArg)                                              \
+  V(InlineCacheMissHandlerTwoArgs)                                             \
+  V(InlineCacheMissHandlerThreeArgs)                                           \
+  V(StaticCallMissHandlerOneArg)                                               \
+  V(StaticCallMissHandlerTwoArgs)                                              \
+  V(Instanceof)                                                                \
+  V(TypeCheck)                                                                 \
+  V(BadTypeError)                                                              \
+  V(NonBoolTypeError)                                                          \
+  V(InstantiateType)                                                           \
+  V(InstantiateTypeArguments)                                                  \
+  V(InvokeClosureNoSuchMethod)                                                 \
+  V(InvokeNoSuchMethodDispatcher)                                              \
+  V(MegamorphicCacheMissHandler)                                               \
+  V(OptimizeInvokedFunction)                                                   \
+  V(TraceICCall)                                                               \
+  V(PatchStaticCall)                                                           \
+  V(ReThrow)                                                                   \
+  V(StackOverflow)                                                             \
+  V(Throw)                                                                     \
+  V(TraceFunctionEntry)                                                        \
+  V(TraceFunctionExit)                                                         \
+  V(DeoptimizeMaterialize)                                                     \
+  V(UpdateFieldCid)                                                            \
+  V(InitStaticField)                                                           \
+  V(GrowRegExpStack)                                                           \
+  V(CompileFunction)                                                           \
+  V(TestSmiSub)                                                                \

[Florian Schneider, 2015/09/01 06:19:05]

We don't want to have test function stored in each thread object.

[rmacnak, 2015/09/01 17:26:54]

True, but if they aren't there we need a special case in emitting runtime calls
that doesn't load from the thread and we are no longer testing the same
mechanism. Or we need a mechanism to only include the test functions in these
lists when building run_vm_tests.

[Florian Schneider, 2015/09/02 08:22:03]

Maybe we can just test runtime invocation using the existing functions and
remove those completely then? It's not a lot of space, but seems like a bad
smell, when we have unused slots for test functions here in a release build.

e.g. invoke AllocateArray or LibcPow instead of calling these test-only
functions.

[rmacnak, 2015/09/02 21:58:18]

Done.

+
+#define LEAF_RUNTIME_ENTRY_LIST(V)                                             \
+  V(void, PrintStopMessage, const char*)                                       \
+  V(intptr_t, DeoptimizeCopyFrame, uword)                                      \
+  V(void, DeoptimizeFillFrame, uword)                                          \
+  V(void, StoreBufferBlockProcess, Thread*)                                    \
+  V(intptr_t, BigintCompare, RawBigint*, RawBigint*)                           \
+  V(double, LibcPow, double, double)                                           \
+  V(double, DartModulo, double, double)                                        \
+  V(double, LibcFloor, double)                                                 \
+  V(double, LibcCeil, double)                                                  \
+  V(double, LibcTrunc, double)                                                 \
+  V(double, LibcRound, double)                                                 \
+  V(double, LibcCos, double)                                                   \
+  V(double, LibcSin, double)                                                   \
+  V(RawBool*, CaseInsensitiveCompareUC16,                                      \
+    RawString*, RawSmi*, RawSmi*, RawSmi*)                                     \
+  V(RawObject*, TestLeafSmiAdd, RawObject*, RawObject*)                        \

[Florian Schneider, 2015/09/01 06:19:05]

Same here.

+
+}  // namespace dart
+
+#endif  // VM_RUNTIME_ENTRY_LIST_H_