X87: Skip MultiReturnSelect_f32/MultiReturnSelect_f64 tests.

X87: Skip MultiReturnSelect_f32/MultiReturnSelect_f64 tests.

  The CL #39795 (https://codereview.chromium.org/2345593003 ) added MultiReturnSelect_f32/MultiReturnSelect_f64 tests.
  Because those tests need 2 float point return registers and x87 only has 1 FP return register, Those tests failed at x87 port.

  This CL skips MultiReturnSelect_f32/MultiReturnSelect_f64 tests for x87.

BUG=

Committed: https://crrev.com/b600ea0672d300e3d32df873c1e97c5977919083
Cr-Commit-Position: refs/heads/master@{#40113}

[zhengxing.li, 2016-10-10 02:33:45]

zhengxing.li@intel.com changed reviewers:
+ bmeurer@chromium.org, titzer@chromium.org, weiliang.lin@intel.com

[zhengxing.li, 2016-10-10 02:33:45]

PTAL, thanks!

[titzer, 2016-10-10 08:36:14]

On 2016/10/10 02:33:45, zhengxing.li wrote:
> PTAL, thanks!

This is OK to disable for now, but they should eventually work. We should be
able to return the other floats on the stack.

lgtm

[zhengxing.li, 2016-10-10 09:05:26]

On 2016/10/10 08:36:14, titzer wrote:
> On 2016/10/10 02:33:45, zhengxing.li wrote:
> > PTAL, thanks!
> 
> This is OK to disable for now, but they should eventually work. We should be
> able to return the other floats on the stack.
> 
> lgtm

Thanks, Titzer!

I tested for IA32, if I just use xmm1 as the FP return register, those tests
failed too, It's correct?
Thanks!

[zhengxing.li, 2016-10-10 09:07:22]

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

[commit-bot: I haz the power, 2016-10-10 09:07:28]

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

[titzer, 2016-10-10 09:17:14]

On 2016/10/10 09:05:26, zhengxing.li wrote:
> On 2016/10/10 08:36:14, titzer wrote:
> > On 2016/10/10 02:33:45, zhengxing.li wrote:
> > > PTAL, thanks!
> > 
> > This is OK to disable for now, but they should eventually work. We should be
> > able to return the other floats on the stack.
> > 
> > lgtm
> 
> Thanks, Titzer!
> 
> I tested for IA32, if I just use xmm1 as the FP return register, those tests
> failed too, It's correct?
> Thanks!

Yeah, sounds like there is a cross-platform problem here. Hopefully it's just
alignment. We should probably add tests for many many float return values in
order to flush those bugs out on all platforms.

[commit-bot: I haz the power, 2016-10-10 09:30:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-10 09:30:56]

Description was changed from

==========
X87: Skip MultiReturnSelect_f32/MultiReturnSelect_f64 tests.

  The CL #39795 (https://codereview.chromium.org/2345593003 ) added
MultiReturnSelect_f32/MultiReturnSelect_f64 tests.
  Because those tests need 2 float point return registers and x87 only has 1 FP
return register, Those tests failed at x87 port.

  This CL skips MultiReturnSelect_f32/MultiReturnSelect_f64 tests for x87.

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

to

==========
X87: Skip MultiReturnSelect_f32/MultiReturnSelect_f64 tests.

  The CL #39795 (https://codereview.chromium.org/2345593003 ) added
MultiReturnSelect_f32/MultiReturnSelect_f64 tests.
  Because those tests need 2 float point return registers and x87 only has 1 FP
return register, Those tests failed at x87 port.

  This CL skips MultiReturnSelect_f32/MultiReturnSelect_f64 tests for x87.

BUG=

Committed: https://crrev.com/b600ea0672d300e3d32df873c1e97c5977919083
Cr-Commit-Position: refs/heads/master@{#40113}
==========

[commit-bot: I haz the power, 2016-10-10 09:30:59]

Patchset 1 (id:??) landed as
https://crrev.com/b600ea0672d300e3d32df873c1e97c5977919083
Cr-Commit-Position: refs/heads/master@{#40113}