X87: Change the test case for X87 float operations

X87: Change the test case for X87 float operations

       The CL https://codereview.chromium.org/1409013004 added / changed some test cases.
       Some new test cases use CheckFloatEq(...) and CheckDoubleEq(...) function for result
       check. When GCC compiling the CheckFloatEq() and CheckDoubleEq() function, those inlined
       functions has different behavior comparing with GCC ia32 build and x87 build. The major
       difference is sse float register still has single precision rounding semantic. While X87
       register has no such rounding precsion semantic when directly use register value. The V8
       turbofan JITTed has exactly same result in both X87 and IA32 port.

       So we add the following sentence to do type case to keep the same precision.
          float expect = *i * *j; // *i + *j, etc.

       For test case "RunFloat64MulAndFloat64Add1 / RunFloat64MulAndFloat64Add2 / RunFloat64MulAndFloat64Sub1
       / RunFloat64MulAndFloat64Sub2", the expected result calcaulated by GCC has difference precsion
       when comparing with V8 turbofan result for X87 platform. (Turbofan X87 result is the same as
       IA32 GCC and IA32 Turbofan). So we have to disable those four cases for X87 port.

BUG=

Committed: https://crrev.com/5736eb0ccbae38894cec7dd86633717d718a5eb7
Cr-Commit-Position: refs/heads/master@{#31808}

[chunyang.dai, 2015-11-03 14:51:20]

Description was changed from

==========
X87: Change the test case for X87 float operations

       Some new test cases use CheckFloatEq(...) and CheckDoubleEq(...) function
for result
       check. When GCC compiling the CheckFloatEq() and CheckDoubleEq()
function, those inlined
       functions has different behavior comparing with GCC ia32 build and x87
build. The major
       difference is sse float register still has single precision rounding
semantic. While X87
       register has no such rounding precsion semantic when directly use registe
value. The V8
       turbofan JITTed has exactly same result in both X87 and IA32 port.

       So we add the following sentence to do type case to keep the same
precision.
          float expect = *i * *j; // *i + *j, etc.

       For test case "RunFloat64MulAndFloat64Add1 / RunFloat64MulAndFloat64Add2
/ RunFloat64MulAndFloat64Sub1
       / RunFloat64MulAndFloat64Sub2", the expected result calcaulated by GCC
has difference precsion
       when comparing with V8 turbofan result for X87 platform. (Turbofan X87
result is the same as
       IA32 GCC and IA32 Turbofan). So we have to disable those four cases for
X87 port.

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

to

==========
X87: Change the test case for X87 float operations

       The CL https://codereview.chromium.org/1409013004 added / changed some
test cases.
       Some new test cases use CheckFloatEq(...) and CheckDoubleEq(...) function
for result
       check. When GCC compiling the CheckFloatEq() and CheckDoubleEq()
function, those inlined
       functions has different behavior comparing with GCC ia32 build and x87
build. The major
       difference is sse float register still has single precision rounding
semantic. While X87
       register has no such rounding precsion semantic when directly use
register value. The V8
       turbofan JITTed has exactly same result in both X87 and IA32 port.

       So we add the following sentence to do type case to keep the same
precision.
          float expect = *i * *j; // *i + *j, etc.

       For test case "RunFloat64MulAndFloat64Add1 / RunFloat64MulAndFloat64Add2
/ RunFloat64MulAndFloat64Sub1
       / RunFloat64MulAndFloat64Sub2", the expected result calcaulated by GCC
has difference precsion
       when comparing with V8 turbofan result for X87 platform. (Turbofan X87
result is the same as
       IA32 GCC and IA32 Turbofan). So we have to disable those four cases for
X87 port.

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

[chunyang.dai, 2015-11-03 14:52:34]

chunyang.dai@intel.com changed reviewers:
+ ahaas@chromium.org, titzer@chromium.org, weiliang.lin@intel.com

[chunyang.dai, 2015-11-03 14:52:35]

PTAL.

[titzer, 2015-11-03 19:01:43]

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

https://codereview.chromium.org/1430943002/diff/1/test/cctest/compiler/test-r...
test/cctest/compiler/test-run-machops.cc:3246: float expected = *i + *j;
These should probably be volatile float, as well.

[chunyang.dai, 2015-11-04 08:28:02]

Hi, Titizer.
I updated it according to your suggestion.
PTAL.

thanks.

[titzer, 2015-11-04 18:07:54]

lgtm

[chunyang.dai, 2015-11-04 23:28:26]

The CQ bit was checked by chunyang.dai@intel.com

[commit-bot: I haz the power, 2015-11-04 23:28:33]

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

[commit-bot: I haz the power, 2015-11-05 00:14:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-05 00:15:45]

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