[turbofan] Handle void returns in instruction selector.

[turbofan] Handle void returns in instruction selector.

R=jarin@chromium.org
BUG=

Committed: https://crrev.com/6b63aa06d31c3ca8d24c61165491636f977d90c1
Cr-Commit-Position: refs/heads/master@{#29999}

[Jarin, 2015-08-04 12:38:30]

lgtm

https://codereview.chromium.org/1269183002/diff/1/src/compiler/instruction-se...
File src/compiler/instruction-selector.cc (right):

https://codereview.chromium.org/1269183002/diff/1/src/compiler/instruction-se...
src/compiler/instruction-selector.cc:1004: Emit(kArchRet, g.NoOutput(),
g.Use(value));
Any reason for calling g.Use(value)? This might keep that value alive for longer
than it needs to be.

[titzer, 2015-08-04 12:49:15]

On 2015/08/04 12:38:30, Jarin wrote:
> lgtm
> 
>
https://codereview.chromium.org/1269183002/diff/1/src/compiler/instruction-se...
> File src/compiler/instruction-selector.cc (right):
> 
>
https://codereview.chromium.org/1269183002/diff/1/src/compiler/instruction-se...
> src/compiler/instruction-selector.cc:1004: Emit(kArchRet, g.NoOutput(),
> g.Use(value));
> Any reason for calling g.Use(value)? This might keep that value alive for
longer
> than it needs to be.

Fixed.

[titzer, 2015-08-04 12:49:20]

The CQ bit was checked by titzer@chromium.org

[titzer, 2015-08-04 12:49:21]

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

[commit-bot: I haz the power, 2015-08-04 12:49:26]

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

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-04 13:14:22]

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