[fullcodegen] Factor out EmitNamedPropertyLoad.

src/ia32/macro-assembler-ia32.h

Index: src/ia32/macro-assembler-ia32.h
diff --git a/src/ia32/macro-assembler-ia32.h b/src/ia32/macro-assembler-ia32.h
index 9ebae1f463d6d8192d4b0719601cbc97f118f80c..31a89ba79ad85074911e9f229615a1543ed26082 100644
--- a/src/ia32/macro-assembler-ia32.h
+++ b/src/ia32/macro-assembler-ia32.h
@@ -796,6 +796,7 @@ class MacroAssembler: public Assembler {
   void Move(XMMRegister dst, float src) { Move(dst, bit_cast<uint32_t>(src)); }
   void Move(XMMRegister dst, double src) { Move(dst, bit_cast<uint64_t>(src)); }
 
+  void Move(Register dst, Handle<Object> x) { mov(dst, Immediate(x)); }

[Michael Starzinger, 2016/02/24 13:28:51]

I am using the "mov" here that can only deal with handles referencing a
HeapObject in old-space. On some architectures the existing "Move" is more
generic. We could delegate to "LoadObject" to have this be the generic version.
Either is fine with me. Please advise which is preferred here.

[Benedikt Meurer, 2016/02/24 13:38:30]

I don't care. Either is fine with me.

[Igor Sheludko, 2016/02/24 13:56:46]

I think we should make it safe for any kind of handles, like on x64 for example,
otherwise it would be so easy to misuse it during poring x64 change to ia32.

[Michael Starzinger, 2016/02/24 17:13:58]

Done. Thanks for the data point. Done for all architecture where it applies
(i.e. ARM64, ia32, MIPS, MIPS64 and x87).

   void Move(Register dst, Smi* source) { Move(dst, Immediate(source)); }
 
   // Push a handle value.