Remove manual representation annotations for constants.

Remove manual representation annotations for constants.

R=mstarzinger@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=15130

[Toon Verwaest, 2013-06-13 17:02:27]

PTAL

[Michael Starzinger, 2013-06-13 17:33:56]

LGTM with nits.

https://codereview.chromium.org/16818016/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/16818016/diff/1/src/hydrogen.cc#newcode1415
src/hydrogen.cc:1415:
Handle<FixedArray>(isolate()->heap()->empty_fixed_array()));
I know this is not you change, but can we use
"isolate()->factory()->empty_fixed_array()" here instead?

https://codereview.chromium.org/16818016/diff/1/src/hydrogen.cc#newcode8338
src/hydrogen.cc:8338: HConstant* double_one = new(zone()) HConstant(1);
Can we use Graph::GetConstant1() here instead?

[Toon Verwaest, 2013-06-13 17:37:56]

Addressed comments.

https://codereview.chromium.org/16818016/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/16818016/diff/1/src/hydrogen.cc#newcode1415
src/hydrogen.cc:1415:
Handle<FixedArray>(isolate()->heap()->empty_fixed_array()));
On 2013/06/13 17:33:56, Michael Starzinger wrote:
> I know this is not you change, but can we use
> "isolate()->factory()->empty_fixed_array()" here instead?

Done.

https://codereview.chromium.org/16818016/diff/1/src/hydrogen.cc#newcode8338
src/hydrogen.cc:8338: HConstant* double_one = new(zone()) HConstant(1);
On 2013/06/13 17:33:56, Michael Starzinger wrote:
> Can we use Graph::GetConstant1() here instead?

Done.

[Toon Verwaest, 2013-06-13 17:38:18]

Committed patchset #2 manually as r15130 (presubmit successful).