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

Issue 2639693002: VM: Fix umulh on simarm64 Windows. (Closed)

Created:
3 years, 11 months ago by Florian Schneider
Modified:
3 years, 11 months ago
Reviewers:
zra
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

VM: Fix umulh on simarm64 Windows. Also add correct suffix for uint64_t in another place. BUG= R=zra@google.com Committed: https://github.com/dart-lang/sdk/commit/13383d7f85b96b75d2c190b4dabf05fe0b653165

Patch Set 1 #

Total comments: 2

Patch Set 2 : add unsigned #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -6 lines) Patch
M runtime/vm/simulator_arm64.cc View 1 2 chunks +7 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Florian Schneider
3 years, 11 months ago (2017-01-17 20:11:13 UTC) #2
zra
lgtm with question https://codereview.chromium.org/2639693002/diff/1/runtime/vm/simulator_arm64.cc File runtime/vm/simulator_arm64.cc (right): https://codereview.chromium.org/2639693002/diff/1/runtime/vm/simulator_arm64.cc#newcode2681 runtime/vm/simulator_arm64.cc:2681: static_cast<__int128>(rn_val) * static_cast<__int128>(rm_val); Is this one ...
3 years, 11 months ago (2017-01-17 20:18:05 UTC) #3
Florian Schneider
https://codereview.chromium.org/2639693002/diff/1/runtime/vm/simulator_arm64.cc File runtime/vm/simulator_arm64.cc (right): https://codereview.chromium.org/2639693002/diff/1/runtime/vm/simulator_arm64.cc#newcode2681 runtime/vm/simulator_arm64.cc:2681: static_cast<__int128>(rn_val) * static_cast<__int128>(rm_val); On 2017/01/17 20:18:04, zra wrote: > ...
3 years, 11 months ago (2017-01-17 21:16:47 UTC) #4
Florian Schneider
3 years, 11 months ago (2017-01-17 21:23:52 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
13383d7f85b96b75d2c190b4dabf05fe0b653165 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698