Revision: 3649 Author: [email protected] Date: Tue Jan 19 05:36:12 2010 Log: Fix bug in X64 code on trunk. See Chromium issue 32436. (Fixed on bleeding edge in revision 3644).
Review URL: http://codereview.chromium.org:80/552041 http://code.google.com/p/v8/source/detail?r=3649 Modified: /trunk/src/version.cc /trunk/src/x64/macro-assembler-x64.cc ======================================= --- /trunk/src/version.cc Thu Jan 14 07:28:53 2010 +++ /trunk/src/version.cc Tue Jan 19 05:36:12 2010 @@ -35,7 +35,7 @@ #define MAJOR_VERSION 2 #define MINOR_VERSION 0 #define BUILD_NUMBER 6 -#define PATCH_LEVEL 0 +#define PATCH_LEVEL 1 #define CANDIDATE_VERSION false // Define SONAME to have the SCons build the put a specific SONAME into the ======================================= --- /trunk/src/x64/macro-assembler-x64.cc Wed Dec 16 07:36:05 2009 +++ /trunk/src/x64/macro-assembler-x64.cc Tue Jan 19 05:36:12 2010 @@ -2374,7 +2374,7 @@ // Move up the chain of contexts to the context containing the slot. movq(dst, Operand(rsi, Context::SlotOffset(Context::CLOSURE_INDEX))); // Load the function context (which is the incoming, outer context). - movq(rax, FieldOperand(rax, JSFunction::kContextOffset)); + movq(dst, FieldOperand(dst, JSFunction::kContextOffset)); for (int i = 1; i < context_chain_length; i++) { movq(dst, Operand(dst, Context::SlotOffset(Context::CLOSURE_INDEX))); movq(dst, FieldOperand(dst, JSFunction::kContextOffset));
-- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
