X64 Crankshaft: Reapply reverted operations with DoLoadGlobal disabled.

X64 Crankshaft: Reapply reverted operations with DoLoadGlobal disabled.

Committed: http://code.google.com/p/v8/source/detail?r=6510

[Rico, 2011-01-27 08:23:02]

LGTM, maybe we should add:
ASSERT(code->safepoint_table_start() - last_pc_offset > 0);
To the deoptimizer-x64 (and ia32+arm for that sake) before assigning this to an
unsigned integer.

http://codereview.chromium.org/6397002/diff/1/src/x64/lithium-codegen-x64.cc
File src/x64/lithium-codegen-x64.cc (right):

http://codereview.chromium.org/6397002/diff/1/src/x64/lithium-codegen-x64.cc#...
src/x64/lithium-codegen-x64.cc:1388: Abort("Sometimes triggers bug in
deoptimization. Keep disabled for now.");
With the fix for the conditional jump to a byte* we should be able to enable
this method as well.

[Rico, 2011-01-27 11:52:40]

LGTM

http://codereview.chromium.org/6397002/diff/4001/src/x64/macro-assembler-x64.cc
File src/x64/macro-assembler-x64.cc (right):

http://codereview.chromium.org/6397002/diff/4001/src/x64/macro-assembler-x64....
src/x64/macro-assembler-x64.cc:1773: if (V8::UseCrankshaft()) {
Maybe add the comment from ia32 stating why we need to do the extra indirection

[Lasse Reichstein, 2011-01-27 11:57:33]

http://codereview.chromium.org/6397002/diff/4001/src/x64/macro-assembler-x64.cc
File src/x64/macro-assembler-x64.cc (right):

http://codereview.chromium.org/6397002/diff/4001/src/x64/macro-assembler-x64....
src/x64/macro-assembler-x64.cc:1773: if (V8::UseCrankshaft()) {
A comment has been added (without the TODO and "for now") part.