[turbofan] NaN is never truish.

[turbofan] NaN is never truish.

BUG=v8:4207
LOG=y
R=jkummerow@chromium.org

Committed: https://crrev.com/78e9a2dff20f9f16c69271acfd2d348cd78df816
Cr-Commit-Position: refs/heads/master@{#29230}

[Benedikt Meurer, 2015-06-23 11:35:28]

Hey Jakob,

Trivial fix for the NaN truish issue. Please take a look.

Thanks,
Benedikt

[titzer, 2015-06-23 11:41:27]

titzer@chromium.org changed reviewers:
+ titzer@chromium.org

[titzer, 2015-06-23 11:41:27]

https://codereview.chromium.org/1198993009/diff/1/test/mjsunit/compiler/regre...
File test/mjsunit/compiler/regress-4207.js (right):

https://codereview.chromium.org/1198993009/diff/1/test/mjsunit/compiler/regre...
test/mjsunit/compiler/regress-4207.js:7: function foo() { return 0/0 || 1; }
What about adding a test for && as well?

[Benedikt Meurer, 2015-06-23 11:58:53]

https://codereview.chromium.org/1198993009/diff/1/test/mjsunit/compiler/regre...
File test/mjsunit/compiler/regress-4207.js (right):

https://codereview.chromium.org/1198993009/diff/1/test/mjsunit/compiler/regre...
test/mjsunit/compiler/regress-4207.js:7: function foo() { return 0/0 || 1; }
On 2015/06/23 11:41:27, titzer wrote:
> What about adding a test for && as well?

Done.

[titzer, 2015-06-23 12:02:45]

lgtm

[Benedikt Meurer, 2015-06-23 12:02:56]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-06-23 12:03:03]

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

[commit-bot: I haz the power, 2015-06-23 12:24:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-23 12:25:01]

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