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

Issue 622713003: Skip test on all architectures but arm. (Closed)

Created:
6 years, 2 months ago by Michael Achenbach
Modified:
6 years, 2 months ago
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

Skip test on all architectures but arm. BUG= R=bmeurer@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=24385

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M test/mjsunit/mjsunit.status View 1 chunk +3 lines, -1 line 2 comments Download

Messages

Total messages: 6 (2 generated)
Michael Achenbach
PTAL
6 years, 2 months ago (2014-10-02 07:28:51 UTC) #2
Benedikt Meurer
lgtm
6 years, 2 months ago (2014-10-02 08:17:39 UTC) #3
Michael Achenbach
Committed patchset #1 (id:1) manually as 24385 (presubmit successful).
6 years, 2 months ago (2014-10-02 08:38:03 UTC) #4
Jakob Kummerow
6 years, 2 months ago (2014-10-02 09:00:25 UTC) #6
Message was sent while issue was closed.
DBC.

https://codereview.chromium.org/622713003/diff/1/test/mjsunit/mjsunit.status
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/622713003/diff/1/test/mjsunit/mjsunit.status#...
test/mjsunit/mjsunit.status:199: 'nans': [PASS, ],
wut? Just remove the expectation! Or was this an accidental edit?

https://codereview.chromium.org/622713003/diff/1/test/mjsunit/mjsunit.status#...
test/mjsunit/mjsunit.status:202: 'math-floor-of-div-nosudiv': [PASS, SLOW,
['arch not in [arm, arm64, android_arm, android_arm64]', SKIP]],
AFAIK sdiv/udiv are always available on arm64, so the list should be [arm,
android_arm].

Powered by Google App Engine
This is Rietveld 408576698