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

Issue 2992483002: [vm] Update runtime/vm/stub_code_<arch>_test to avoid using Bigints (Closed)

Created:
3 years, 5 months ago by alexmarkov
Modified:
3 years, 4 months ago
Reviewers:
zra, rmacnak, siva
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

[vm] Update runtime/vm/stub_code_<arch>_test to avoid using Bigints In preparation for limiting Dart integers to 64 bits, test for stub code generation is updated to avoid using Bigints. Instead of calling kBigintCompareRuntimeEntry leaf runtime entry, this test now calls kCaseInsensitiveCompareUC16RuntimeEntry. R=zra@google.com Issue: https://github.com/dart-lang/sdk/issues/30103 Committed: https://github.com/dart-lang/sdk/commit/da654d9a5f387e54d7a991df7d3ed19b5230d8cf

Patch Set 1 #

Total comments: 2

Patch Set 2 : Cleanup: _assembler_ locals are renamed to assembler. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+109 lines, -83 lines) Patch
M runtime/vm/stub_code_arm64_test.cc View 1 3 chunks +26 lines, -20 lines 0 comments Download
M runtime/vm/stub_code_arm_test.cc View 1 3 chunks +26 lines, -20 lines 0 comments Download
M runtime/vm/stub_code_ia32_test.cc View 1 3 chunks +31 lines, -23 lines 0 comments Download
M runtime/vm/stub_code_x64_test.cc View 1 3 chunks +26 lines, -20 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
alexmarkov
3 years, 5 months ago (2017-07-25 23:10:19 UTC) #2
zra
lgtm w/ cleanup suggestion. https://codereview.chromium.org/2992483002/diff/1/runtime/vm/stub_code_arm64_test.cc File runtime/vm/stub_code_arm64_test.cc (right): https://codereview.chromium.org/2992483002/diff/1/runtime/vm/stub_code_arm64_test.cc#newcode97 runtime/vm/stub_code_arm64_test.cc:97: Assembler _assembler_; It seems like ...
3 years, 4 months ago (2017-07-26 16:12:23 UTC) #3
alexmarkov
https://codereview.chromium.org/2992483002/diff/1/runtime/vm/stub_code_arm64_test.cc File runtime/vm/stub_code_arm64_test.cc (right): https://codereview.chromium.org/2992483002/diff/1/runtime/vm/stub_code_arm64_test.cc#newcode97 runtime/vm/stub_code_arm64_test.cc:97: Assembler _assembler_; On 2017/07/26 16:12:23, zra wrote: > It ...
3 years, 4 months ago (2017-07-26 20:14:14 UTC) #4
alexmarkov
3 years, 4 months ago (2017-07-26 20:15:14 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
da654d9a5f387e54d7a991df7d3ed19b5230d8cf (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698