Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Issue 9540010: Update test262 expectations concerning 64-bit precision double. (Closed)

Created:
8 years, 9 months ago by Yang
Modified:
8 years, 9 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev
Visibility:
Public.

Description

Update test262 expectations concerning 64-bit precision double. BUG= TEST= Committed: https://code.google.com/p/v8/source/detail?r=10883

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fixed wrong report line for "PASS, FAIL_OK if <condition>" when expected to fail. #

Patch Set 3 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -5 lines) Patch
M test/test262/test262.status View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M tools/test.py View 1 2 5 chunks +9 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Yang
PTAL.
8 years, 9 months ago (2012-02-29 16:25:15 UTC) #1
Michael Starzinger
https://chromiumcodereview.appspot.com/9540010/diff/1/test/test262/test262.status File test/test262/test262.status (right): https://chromiumcodereview.appspot.com/9540010/diff/1/test/test262/test262.status#newcode74 test/test262/test262.status:74: S8.5_A2.1: PASS, FAIL if ($system == linux && ($arch ...
8 years, 9 months ago (2012-02-29 16:33:16 UTC) #2
Yang
On 2012/02/29 16:33:16, Michael Starzinger wrote: > https://chromiumcodereview.appspot.com/9540010/diff/1/test/test262/test262.status > File test/test262/test262.status (right): > > https://chromiumcodereview.appspot.com/9540010/diff/1/test/test262/test262.status#newcode74 ...
8 years, 9 months ago (2012-03-01 10:58:26 UTC) #3
Michael Starzinger
8 years, 9 months ago (2012-03-01 11:36:28 UTC) #4
LGTM.

Powered by Google App Engine
This is Rietveld 408576698