X87: Skip SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64 tests.

X87: Skip SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64 tests.

  The CL #42545 (https://codereview.chromium.org/2639353002 ) add SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64 tests.
  Those tests failed at x87 port as X87 FPU hardware will convert a sNaN to qNaN automatically.

  This CL skips SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64 tests for x87.

BUG=

Review-Url: https://codereview.chromium.org/2649843002
Cr-Commit-Position: refs/heads/master@{#42591}
Committed: https://chromium.googlesource.com/v8/v8/+/6435c2343ccaf928b0ecb9dedbcea65394e8b037

[zhengxing.li, 2017-01-23 07:13:16]

zhengxing.li@intel.com changed reviewers:
+ ahaas@chromium.org, titzer@chromium.org

[zhengxing.li, 2017-01-23 07:13:16]

PTAL, thanks!

[zhengxing.li, 2017-01-23 07:14:50]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai@intel.com

[zhengxing.li, 2017-01-23 07:14:51]

PTAL, thanks!

[ahaas, 2017-01-23 08:18:10]

lgtm

[zhengxing.li, 2017-01-23 08:39:49]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2017-01-23 08:40:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chunyang.dai, 2017-01-23 08:41:54]

On 2017/01/23 08:40:00, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

LGTM.

thanks

[commit-bot: I haz the power, 2017-01-23 09:10:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485160789405640, "parent_rev":
"96220730e688f01909ad61f437d1a8b88a58d173", "commit_rev":
"6435c2343ccaf928b0ecb9dedbcea65394e8b037"}

[commit-bot: I haz the power, 2017-01-23 09:10:58]

Description was changed from

==========
X87: Skip
SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64
tests.

  The CL #42545 (https://codereview.chromium.org/2639353002 ) add
SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64
tests.
  Those tests failed at x87 port as X87 FPU hardware will convert a sNaN to qNaN
automatically.

  This CL skips
SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64
tests for x87.

BUG=
==========

to

==========
X87: Skip
SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64
tests.

  The CL #42545 (https://codereview.chromium.org/2639353002 ) add
SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64
tests.
  Those tests failed at x87 port as X87 FPU hardware will convert a sNaN to qNaN
automatically.

  This CL skips
SignallingNanSurvivesI32ReinterpretF32/SignallingNanSurvivesI64ReinterpretF64
tests for x87.

BUG=

Review-Url: https://codereview.chromium.org/2649843002
Cr-Commit-Position: refs/heads/master@{#42591}
Committed:
https://chromium.googlesource.com/v8/v8/+/6435c2343ccaf928b0ecb9dedbcea65394e...
==========

[commit-bot: I haz the power, 2017-01-23 09:10:59]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/6435c2343ccaf928b0ecb9dedbcea65394e...

[titzer, 2017-01-23 09:23:35]

On 2017/01/23 09:10:59, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/v8/v8/+/6435c2343ccaf928b0ecb9dedbcea65394e...

Any insight on how to fix this correctly on x87? It's part of the WASM spec.

[zhengxing.li, 2017-01-23 09:36:33]

On 2017/01/23 09:23:35, titzer wrote:
> On 2017/01/23 09:10:59, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1) as
> >
>
https://chromium.googlesource.com/v8/v8/+/6435c2343ccaf928b0ecb9dedbcea65394e...
> 
> Any insight on how to fix this correctly on x87? It's part of the WASM spec.

Hi, Titzer:

This issue bothers me for a long time. But I have no good way to fix it in a
short time. I will still follow this issue and fix it once I get a good idea
about it.

Thanks!

[titzer, 2017-01-23 10:11:48]

On 2017/01/23 09:36:33, zhengxing.li wrote:
> On 2017/01/23 09:23:35, titzer wrote:
> > On 2017/01/23 09:10:59, commit-bot: I haz the power wrote:
> > > Committed patchset #1 (id:1) as
> > >
> >
>
https://chromium.googlesource.com/v8/v8/+/6435c2343ccaf928b0ecb9dedbcea65394e...
> > 
> > Any insight on how to fix this correctly on x87? It's part of the WASM spec.
> 
> Hi, Titzer:
> 
> This issue bothers me for a long time. But I have no good way to fix it in a
> short time. I will still follow this issue and fix it once I get a good idea
> about it.
> 
> Thanks!

Thanks, as long as it is on your radar.