Reviewers: Jakob,

Description:
Merged r15577 into 3.19 branch.

MIPS: Do not omit the write-barrier if the input value is a smi.

BUG=chromium:254570

[email protected]


Please review this at https://codereview.chromium.org/19779002/

SVN Base: https://v8.googlecode.com/svn/branches/3.19

Affected files:
  M src/mips/stub-cache-mips.cc
  M src/version.cc


Index: src/mips/stub-cache-mips.cc
diff --git a/src/mips/stub-cache-mips.cc b/src/mips/stub-cache-mips.cc
index 51df45224abdb33a3d2e1d7cf4d758cf06b6e2d9..786132fd56450b698b45c9cbaa15b88cad2a3b06 100644
--- a/src/mips/stub-cache-mips.cc
+++ b/src/mips/stub-cache-mips.cc
@@ -594,9 +594,6 @@ void StubCompiler::GenerateStoreTransition(MacroAssembler* masm,
     }

     if (!FLAG_track_fields || !representation.IsSmi()) {
-      // Skip updating write barrier if storing a smi.
-      __ JumpIfSmi(value_reg, &exit);
-
       // Update the write barrier for the array address.
       // Pass the now unused name_reg as a scratch register.
       if (!FLAG_track_double_fields || !representation.IsDouble()) {
@@ -626,9 +623,6 @@ void StubCompiler::GenerateStoreTransition(MacroAssembler* masm,
     }

     if (!FLAG_track_fields || !representation.IsSmi()) {
-      // Skip updating write barrier if storing a smi.
-      __ JumpIfSmi(value_reg, &exit);
-
       // Update the write barrier for the array address.
       // Ok to clobber receiver_reg and name_reg, since we return.
       if (!FLAG_track_double_fields || !representation.IsDouble()) {
Index: src/version.cc
diff --git a/src/version.cc b/src/version.cc
index 985495609dc03b7f8150809ba4f0e27bf092e566..683186178c1b711dd7b85771f1f1d6d6adfde1f1 100644
--- a/src/version.cc
+++ b/src/version.cc
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     19
 #define BUILD_NUMBER      18
-#define PATCH_LEVEL       12
+#define PATCH_LEVEL       13
 // Use 1 for candidates and 0 otherwise.
 // (Boolean macro values are not supported by all preprocessors.)
 #define IS_CANDIDATE_VERSION 0


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to