Revision: 19508
Author:   [email protected]
Date:     Thu Feb 20 12:33:58 2014 UTC
Log:      Merged r19500 into 3.23 branch.

MIPS: Fix performance regression on sunspider string-unpack-code.

[email protected]
BUG=

Review URL: https://codereview.chromium.org/173703003
http://code.google.com/p/v8/source/detail?r=19508

Modified:
 /branches/3.23/src/mips/stub-cache-mips.cc
 /branches/3.23/src/version.cc

=======================================
--- /branches/3.23/src/mips/stub-cache-mips.cc  Wed Jan 15 18:11:24 2014 UTC
+++ /branches/3.23/src/mips/stub-cache-mips.cc  Thu Feb 20 12:33:58 2014 UTC
@@ -2778,10 +2778,11 @@
   }

   Label number_case;
+  Register match = scratch1();
   Label* smi_target = IncludesNumberType(types) ? &number_case : &miss;
-  __ JumpIfSmi(receiver(), smi_target);
+  __ JumpIfSmi(receiver(), smi_target, match);  // Reg match is 0 if Smi.

-  Register map_reg = scratch1();
+  Register map_reg = scratch2();

   int receiver_count = types->length();
   int number_of_handled_maps = 0;
@@ -2791,12 +2792,15 @@
     Handle<Map> map = IC::TypeToMap(*type, isolate());
     if (!map->is_deprecated()) {
       number_of_handled_maps++;
+      // Check map and tail call if there's a match.
+ // Separate compare from branch, to provide path for above JumpIfSmi().
+      __ Subu(match, map_reg, Operand(map));
       if (type->Is(Type::Number())) {
         ASSERT(!number_case.is_unused());
         __ bind(&number_case);
       }
       __ Jump(handlers->at(current), RelocInfo::CODE_TARGET,
-          eq, map_reg, Operand(map));
+          eq, match, Operand(zero_reg));
     }
   }
   ASSERT(number_of_handled_maps != 0);
=======================================
--- /branches/3.23/src/version.cc       Wed Feb 12 16:03:10 2014 UTC
+++ /branches/3.23/src/version.cc       Thu Feb 20 12:33:58 2014 UTC
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     23
 #define BUILD_NUMBER      17
-#define PATCH_LEVEL       13
+#define PATCH_LEVEL       14
 // 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