Make GCMole happy about Runtime_ForInFoo methods.

src/runtime.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdlib.h>
 #include <limits>
 
 #include "src/v8.h"
 
 #include "src/accessors.h"
@@ skipping 15125 matching lines @@
 
 // TODO(dcarney): remove this function when TurboFan supports it.
 // Takes the object to be iterated over and the result of GetPropertyNamesFast
 // Returns pair (cache_array, cache_type).
 RUNTIME_FUNCTION_RETURN_PAIR(Runtime_ForInInit) {
   SealHandleScope scope(isolate);
   ASSERT(args.length() == 2);
   // This simulates CONVERT_ARG_HANDLE_CHECKED for calls returning pairs.
   // Not worth creating a macro atm as this function should be removed.
   if (!args[0]->IsJSReceiver() || !args[1]->IsObject()) {
-    return MakePair(isolate->ThrowIllegalOperation(),
-                    isolate->heap()->undefined_value());
+    Object* error = isolate->ThrowIllegalOperation();
+    return MakePair(error, isolate->heap()->undefined_value());
   }
   Handle<JSReceiver> object = args.at<JSReceiver>(0);
   Handle<Object> cache_type = args.at<Object>(1);
   if (cache_type->IsMap()) {
     // Enum cache case.
     if (Map::EnumLengthBits::decode(Map::cast(*cache_type)->bit_field3()) ==
         0) {
       // 0 length enum.
       // Can't handle this case in the graph builder,
       // so transform it into the empty fixed array case.
@@ skipping 32 matching lines @@
 //        cache_type from ForInInit,
 //        the current index)
 // Returns pair (array[index], needs_filtering).
 RUNTIME_FUNCTION_RETURN_PAIR(Runtime_ForInNext) {
   SealHandleScope scope(isolate);
   ASSERT(args.length() == 4);
   // This simulates CONVERT_ARG_HANDLE_CHECKED for calls returning pairs.
   // Not worth creating a macro atm as this function should be removed.
   if (!args[0]->IsJSReceiver() || !args[1]->IsFixedArray() ||
       !args[2]->IsObject() || !args[3]->IsSmi()) {
-    return MakePair(isolate->ThrowIllegalOperation(),
-                    isolate->heap()->undefined_value());
+    Object* error = isolate->ThrowIllegalOperation();
+    return MakePair(error, isolate->heap()->undefined_value());
   }
   Handle<JSReceiver> object = args.at<JSReceiver>(0);
   Handle<FixedArray> array = args.at<FixedArray>(1);
   Handle<Object> cache_type = args.at<Object>(2);
   int index = args.smi_at(3);
   // Figure out first if a slow check is needed for this object.
   bool slow_check_needed = false;
   if (cache_type->IsMap()) {
     if (object->map() != Map::cast(*cache_type)) {
       // Object transitioned.  Need slow check.
@@ skipping 400 matching lines @@
   }
   return NULL;
 }
 
 
 const Runtime::Function* Runtime::FunctionForId(Runtime::FunctionId id) {
   return &(kIntrinsicFunctions[static_cast<int>(id)]);
 }
 
 } }  // namespace v8::internal