On 2009/06/23 11:00:57, Lasse Reichstein wrote:
> Could you check the fix?

> http://codereview.chromium.org/146022/diff/1/7
> File src/x64/macro-assembler-x64.cc (right):

> http://codereview.chromium.org/146022/diff/1/7#newcode564
> Line 564: if (!resolved) {
> Attempted fixed. Please check if it makes sense.

Looks right.  Maybe the constant could be the same as the general one
used by the relocator,
   // Distance between the address of the code target in the call
instruction
   // and the return address.  Checked in the debug build.
   static const int Assembler::kTargetAddrToReturnAddrDist = 3 +
kPointerSize;


http://codereview.chromium.org/146022

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

Reply via email to