Don't ignore the actual input representation in UnaryMathOperations

Don't ignore the actual input representation in UnaryMathOperations

R=jkummerow@chromium.org

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

[Toon Verwaest, 2013-06-07 08:26:13]

PTAL

[Jakob Kummerow, 2013-06-07 08:44:45]

https://chromiumcodereview.appspot.com/16035014/diff/1/src/hydrogen-instructi...
File src/hydrogen-instructions.cc (right):

https://chromiumcodereview.appspot.com/16035014/diff/1/src/hydrogen-instructi...
src/hydrogen-instructions.cc:3075: rep =
rep.generalize(observed_input_representation(1));
HUnaryMathOperation doesn't have observed input representations (because these
ops are just calls in full codegen, and call ICs don't observe types of the
call's arguments), it just falls back to the HValue default implementation which
returns None.

[Toon Verwaest, 2013-06-07 09:09:52]

Addressed comment, ptal

https://chromiumcodereview.appspot.com/16035014/diff/1/src/hydrogen-instructi...
File src/hydrogen-instructions.cc (right):

https://chromiumcodereview.appspot.com/16035014/diff/1/src/hydrogen-instructi...
src/hydrogen-instructions.cc:3075: rep =
rep.generalize(observed_input_representation(1));
On 2013/06/07 08:44:45, Jakob wrote:
> HUnaryMathOperation doesn't have observed input representations (because these
> ops are just calls in full codegen, and call ICs don't observe types of the
> call's arguments), it just falls back to the HValue default implementation
which
> returns None.

Done.

[Jakob Kummerow, 2013-06-07 09:10:45]

lgtm

[Toon Verwaest, 2013-06-07 09:11:38]

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