Change return type of FrameDescription::GetFrameSize to avoid unneeded type casts.

src/deoptimizer.cc

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 143 matching lines @@
   // Create the GC safe output frame information and register it for GC
   // handling.
   ASSERT_LT(frame_index, deoptimizer->output_count());
   DeoptimizedFrameInfo* info =
       new DeoptimizedFrameInfo(deoptimizer, frame_index);
   isolate->deoptimizer_data()->deoptimized_frame_info_ = info;
 
   // Get the "simulated" top and size for the requested frame.
   Address top =
       reinterpret_cast<Address>(deoptimizer->output_[frame_index]->GetTop());
-  intptr_t size =
+  unsigned size =
       deoptimizer->output_[frame_index]->GetFrameSize() / kPointerSize;
 
   // Done with the GC-unsafe frame descriptions. This re-enables allocation.
   deoptimizer->DeleteFrameDescriptions();
 
   // Allocate a heap number for the doubles belonging to this frame.
   deoptimizer->MaterializeHeapNumbersForDebuggerInspectableFrame(
       top, size, info);
 
   // Finished using the deoptimizer instance.
@@ skipping 932 matching lines @@
     SetFrameSlot(o, kZapUint32);
   }
 }
 
 
 unsigned FrameDescription::GetOffsetFromSlotIndex(Deoptimizer* deoptimizer,
                                                   int slot_index) {
   if (slot_index >= 0) {
     // Local or spill slots. Skip the fixed part of the frame
     // including all arguments.
-    unsigned base = static_cast<unsigned>(
-        GetFrameSize() - deoptimizer->ComputeFixedSize(GetFunction()));
+    unsigned base =
+        GetFrameSize() - deoptimizer->ComputeFixedSize(GetFunction());
     return base - ((slot_index + 1) * kPointerSize);
   } else {
     // Incoming parameter.
-    unsigned base = static_cast<unsigned>(GetFrameSize() -
+    unsigned base = GetFrameSize() -
         deoptimizer->ComputeIncomingArgumentSize(GetFunction()));
     return base - ((slot_index + 1) * kPointerSize);
   }
 }
 
 
 unsigned FrameDescription::GetExpressionCount(Deoptimizer* deoptimizer) {
   ASSERT_EQ(Code::FUNCTION, kind_);
-  intptr_t size = GetFrameSize() - deoptimizer->ComputeFixedSize(GetFunction());
-  return static_cast<unsigned>(size / kPointerSize);
+  unsigned size = GetFrameSize() - deoptimizer->ComputeFixedSize(GetFunction());
+  return size / kPointerSize;
 }
 
 
 Object* FrameDescription::GetExpression(Deoptimizer* deoptimizer, int index) {
   ASSERT_EQ(Code::FUNCTION, kind_);
   unsigned offset = GetOffsetFromSlotIndex(deoptimizer, index);
   return reinterpret_cast<Object*>(*GetFrameSlotPointer(offset));
 }
 
 
@@ skipping 283 matching lines @@
 DeoptimizedFrameInfo::~DeoptimizedFrameInfo() {
   delete expression_stack_;
 }
 
 void DeoptimizedFrameInfo::Iterate(ObjectVisitor* v) {
   v->VisitPointers(expression_stack_, expression_stack_ + expression_count_);
 }
 
 
 } }  // namespace v8::internal