PPC: Cross-script variables handling fixed. It was possible to write to read-only global variable.

src/ppc/code-stubs-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/base/bits.h"
 #include "src/bootstrapper.h"
@@ skipping 5359 matching lines @@
   for (int i = 0; i < depth(); i++) {
     __ LoadP(context_temp, ContextOperand(context, Context::PREVIOUS_INDEX));
     context = context_temp;
   }
 
   // Load the PropertyCell at the specified slot.
   __ ShiftLeftImm(cell, slot, Operand(kPointerSizeLog2));
   __ add(cell, context, cell);
   __ LoadP(cell, ContextOperand(cell));
 
+  // Check that cell value is not the_hole.
+  __ LoadP(cell_value, FieldMemOperand(cell, PropertyCell::kValueOffset));
+  __ CompareRoot(cell_value, Heap::kTheHoleValueRootIndex);
+  __ beq(&slow_case);
+
   // Load PropertyDetails for the cell (actually only the cell_type and kind).
   __ LoadP(cell_details, FieldMemOperand(cell, PropertyCell::kDetailsOffset));
   __ SmiUntag(cell_details);
   __ andi(cell_details, cell_details,
           Operand(PropertyDetails::PropertyCellTypeField::kMask |
                   PropertyDetails::KindField::kMask));
 
   // Check if PropertyCell holds mutable data.
   Label not_mutable_data;
   __ cmpi(cell_details, Operand(PropertyDetails::PropertyCellTypeField::encode(
                                     PropertyCellType::kMutable) |
                                 PropertyDetails::KindField::encode(kData)));
   __ bne(&not_mutable_data);
   __ JumpIfSmi(value, &fast_smi_case);
 
   __ bind(&fast_heapobject_case);
   __ StoreP(value, FieldMemOperand(cell, PropertyCell::kValueOffset), r0);
   // RecordWriteField clobbers the value register, so we copy it before the
   // call.
   __ mr(r7, value);
   __ RecordWriteField(cell, PropertyCell::kValueOffset, r7, scratch,
                       kLRHasNotBeenSaved, kDontSaveFPRegs, EMIT_REMEMBERED_SET,
                       OMIT_SMI_CHECK);
   __ Ret();
 
   __ bind(&not_mutable_data);
   // Check if PropertyCell value matches the new value (relevant for Constant,
   // ConstantType and Undefined cells).
   Label not_same_value;
-  __ LoadP(cell_value, FieldMemOperand(cell, PropertyCell::kValueOffset));
   __ cmp(cell_value, value);
   __ bne(&not_same_value);
 
   if (FLAG_debug_code) {
     Label done;
     // This can only be true for Constant, ConstantType and Undefined cells,
     // because we never store the_hole via this stub.
     __ cmpi(cell_details,
             Operand(PropertyDetails::PropertyCellTypeField::encode(
                         PropertyCellType::kConstant) |
@@ skipping 406 matching lines @@
                            kStackUnwindSpace, NULL,
                            MemOperand(fp, 6 * kPointerSize), NULL);
 }
 
 
 #undef __
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_TARGET_ARCH_PPC