Fix polymorphic to monomorphic load to take representation into account.

src/code-stubs-hydrogen.cc

Index: src/code-stubs-hydrogen.cc
diff --git a/src/code-stubs-hydrogen.cc b/src/code-stubs-hydrogen.cc
index dbe41cbf5482719be369f92f136f23b094db44a0..16cfd12becb76973aeb541b936794b82f288f332 100644
--- a/src/code-stubs-hydrogen.cc
+++ b/src/code-stubs-hydrogen.cc
@@ -413,7 +413,7 @@ Handle<Code> KeyedLoadFastElementStub::GenerateCode() {
 template<>
 HValue* CodeStubGraphBuilder<LoadFieldStub>::BuildCodeStub() {
   Representation representation = casted_stub()->representation();
-  HInstruction* load = AddInstruction(new(zone()) HLoadNamedField(
+  HInstruction* load = AddInstruction(DoBuildLoadNamedField(
       GetParameter(0), casted_stub()->is_inobject(),
       representation, casted_stub()->offset()));
   return load;
@@ -428,7 +428,7 @@ Handle<Code> LoadFieldStub::GenerateCode() {
 template<>
 HValue* CodeStubGraphBuilder<KeyedLoadFieldStub>::BuildCodeStub() {
   Representation representation = casted_stub()->representation();
-  HInstruction* load = AddInstruction(new(zone()) HLoadNamedField(
+  HInstruction* load = AddInstruction(DoBuildLoadNamedField(

[mvstanton, 2013/05/07 10:17:17]

Since you always add the instruction, maybe the builder should just add it?
(Seems like some builders do the add, others don't.)

[Toon Verwaest, 2013/05/07 10:30:24]

It seems more in sync with the other builds to not add it; so I'll leave it like
it was.

On 2013/05/07 10:17:17, mvstanton wrote:

       GetParameter(0), casted_stub()->is_inobject(),
       representation, casted_stub()->offset()));
   return load;