Add tests for float32/float64 parameters/returns passed in float32/float64 registers.

Add tests for float32/float64 parameters/returns passed in float32/float64 registers.

R=mstarzinger@chromium.org
BUG=

Committed: https://crrev.com/8525136bd3b7e360852bccc5b7f4a6b9e44ea22a
Cr-Commit-Position: refs/heads/master@{#30143}

[Michael Starzinger, 2015-08-12 17:48:41]

LGTM.

https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
File test/cctest/compiler/test-run-native-calls.cc (right):

https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
test/cctest/compiler/test-run-native-calls.cc:33: // Picks a representative set
of integers from the gi
nit: comment looks incomplete.

https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
test/cctest/compiler/test-run-native-calls.cc:76: //    printf("pair = %d,
%d\n", *r0, *r1);
nit: Looks like a leftover.

https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
test/cctest/compiler/test-run-native-calls.cc:406: void
ArgsBuffer<CType>::Mutate() {
Can we just skip the non-specialized implementation? Thereby using it with a new
type parameter will produce a link-time error.

[titzer, 2015-08-13 05:04:47]

On 2015/08/12 17:48:41, Michael Starzinger wrote:
> LGTM.
> 
>
https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
> File test/cctest/compiler/test-run-native-calls.cc (right):
> 
>
https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
> test/cctest/compiler/test-run-native-calls.cc:33: // Picks a representative
set
> of integers from the gi
> nit: comment looks incomplete.

Done.

> 
>
https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
> test/cctest/compiler/test-run-native-calls.cc:76: //    printf("pair = %d,
> %d\n", *r0, *r1);
> nit: Looks like a leftover.
>

Done.
 
>
https://codereview.chromium.org/1291553005/diff/1/test/cctest/compiler/test-r...
> test/cctest/compiler/test-run-native-calls.cc:406: void
> ArgsBuffer<CType>::Mutate() {
> Can we just skip the non-specialized implementation? Thereby using it with a
new
> type parameter will produce a link-time error.

Done.

[titzer, 2015-08-13 05:05:01]

The CQ bit was checked by titzer@chromium.org

[titzer, 2015-08-13 05:05:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1291553005/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-08-13 05:05:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1291553005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1291553005/20001

[commit-bot: I haz the power, 2015-08-13 05:29:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-13 05:30:13]

Patchset 2 (id:??) landed as
https://crrev.com/8525136bd3b7e360852bccc5b7f4a6b9e44ea22a
Cr-Commit-Position: refs/heads/master@{#30143}