PPC: Clean up JSConstructStub

src/ppc/builtins-ppc.cc

 // Copyright 2014 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 "src/v8.h"
 
 #if V8_TARGET_ARCH_PPC
 
 #include "src/codegen.h"
 #include "src/debug.h"
@@ skipping 421 matching lines @@
       if (create_memento) {
         __ addi(r6, r6, Operand(AllocationMemento::kSize / kPointerSize));
       }
 
       __ Allocate(r6, r7, r8, r9, &rt_call, SIZE_IN_WORDS);
 
       // Allocated the JSObject, now initialize the fields. Map is set to
       // initial map and properties and elements are set to empty fixed array.
       // r4: constructor function
       // r5: initial map
-      // r6: object size (not including memento if create_memento)
+      // r6: object size (including memento if create_memento)
       // r7: JSObject (not tagged)
       __ LoadRoot(r9, Heap::kEmptyFixedArrayRootIndex);
       __ mr(r8, r7);
       __ StoreP(r5, MemOperand(r8, JSObject::kMapOffset));
       __ StoreP(r9, MemOperand(r8, JSObject::kPropertiesOffset));
       __ StoreP(r9, MemOperand(r8, JSObject::kElementsOffset));
       __ addi(r8, r8, Operand(JSObject::kElementsOffset + kPointerSize));
 
       __ ShiftLeftImm(r9, r6, Operand(kPointerSizeLog2));
       __ add(r9, r7, r9);  // End of object.
@@ skipping 56 matching lines @@
         __ InitializeFieldsWithFiller(r8, r9, r10);
       }
 
       // Add the object tag to make the JSObject real, so that we can continue
       // and jump into the continuation code at any time from now on. Any
       // failures need to undo the allocation, so that the heap is in a
       // consistent state and verifiable.
       __ addi(r7, r7, Operand(kHeapObjectTag));
 
       // Check if a non-empty properties array is needed. Continue with
-      // allocated object if not fall through to runtime call if it is.
+      // allocated object if not; allocate and initialize a FixedArray if yes.
       // r4: constructor function
       // r7: JSObject
       // r8: start of next object (not tagged)
       __ lbz(r6, FieldMemOperand(r5, Map::kUnusedPropertyFieldsOffset));
       // The field instance sizes contains both pre-allocated property fields
       // and in-object properties.
       __ lbz(r0, FieldMemOperand(r5, Map::kPreAllocatedPropertyFieldsOffset));
       __ add(r6, r6, r0);
       __ lbz(r0, FieldMemOperand(r5, Map::kInObjectPropertiesOffset));
       __ sub(r6, r6, r0, LeaveOE, SetRC);
@@ skipping 1342 matching lines @@
     __ bkpt(0);
   }
 }
 
 
 #undef __
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_TARGET_ARCH_PPC