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

Issue 13584006: Turn off FindCodeObject for simmips. (Closed)

Created:
7 years, 8 months ago by zra
Modified:
7 years, 8 months ago
Reviewers:
regis
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Turn off FindCodeObject for simmips. I hadn't run it with type checks, and so I've got some more work to do on it. Committed: https://code.google.com/p/dart/source/detail?r=20896

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M runtime/tests/vm/vm.status View 1 chunk +1 line, -0 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
zra
7 years, 8 months ago (2013-04-03 21:58:09 UTC) #1
zra
Committed patchset #1 manually as r20896 (presubmit successful).
7 years, 8 months ago (2013-04-03 21:58:49 UTC) #2
regis
7 years, 8 months ago (2013-04-03 22:20:11 UTC) #3
Message was sent while issue was closed.
LGTM with comment for next cl.

https://codereview.chromium.org/13584006/diff/1/runtime/tests/vm/vm.status
File runtime/tests/vm/vm.status (right):

https://codereview.chromium.org/13584006/diff/1/runtime/tests/vm/vm.status#ne...
runtime/tests/vm/vm.status:75: cc/FindCodeObject: Skip
For next cl: skip only in checked mode, since it works in production mode:

[ $arch == simmips && $checked ]
cc/FindCodeObject: Skip

Powered by Google App Engine
This is Rietveld 408576698