Revision: 12890
Author:   [email protected]
Date:     Wed Nov  7 13:38:46 2012
Log:      Move vldr in DoLoadKeyedFastDoubleElement before the hole check
This reschedules instructions to better absorb VFP load latency.
BUG=none
TEST=none

Review URL: https://codereview.chromium.org/11363086
Patch from Rajeev Krithivasan <[email protected]>.
http://code.google.com/p/v8/source/detail?r=12890

Modified:
 /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc

=======================================
--- /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Mon Nov 5 05:28:10 2012 +++ /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Wed Nov 7 13:38:46 2012
@@ -3040,13 +3040,12 @@
                    (instr->additional_index() << element_size_shift)));
   }

+  __ vldr(result, elements, 0);
   if (instr->hydrogen()->RequiresHoleCheck()) {
     __ ldr(scratch, MemOperand(elements, sizeof(kHoleNanLower32)));
     __ cmp(scratch, Operand(kHoleNanUpper32));
     DeoptimizeIf(eq, instr->environment());
   }
-
-  __ vldr(result, elements, 0);
 }


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to