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

Issue 1318003003: VM: Fix bug in ARM v5/v6 loading of immediates. (Closed)

Created:
5 years, 3 months ago by Florian Schneider
Modified:
5 years, 3 months ago
Reviewers:
regis
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

VM: Fix bug in ARM v5/v6 loading of immediates. The condition was not correctly passed on as argument in LoadDecodableImmediate. BUG= R=regis@google.com Committed: https://github.com/dart-lang/sdk/commit/1907b5a3e1df8331317491dc8a831843c7cf6ec0

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -3 lines) Patch
M runtime/vm/assembler_arm.h View 2 chunks +2 lines, -1 line 0 comments Download
M runtime/vm/assembler_arm.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Florian Schneider
5 years, 3 months ago (2015-08-28 10:59:08 UTC) #2
regis
LGTM
5 years, 3 months ago (2015-08-28 16:44:45 UTC) #3
Florian Schneider
5 years, 3 months ago (2015-09-11 08:00:27 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
1907b5a3e1df8331317491dc8a831843c7cf6ec0 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698