Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ dartfuzz ] AOT-DebugX64C - JIT-SLOWPATH-ReleaseSIMARM64C Divergence #45674

Closed
bkonyi opened this issue Apr 13, 2021 · 0 comments
Closed

[ dartfuzz ] AOT-DebugX64C - JIT-SLOWPATH-ReleaseSIMARM64C Divergence #45674

bkonyi opened this issue Apr 13, 2021 · 0 comments
Assignees
@rmacnak-google
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. dartfuzz Found with Dart fuzzing (DartFuzz, libFuzzer, etc.)

Comments

@bkonyi
Copy link
Contributor

bkonyi commented Apr 13, 2021

Full logs.

-- BEGIN REPRODUCE  --

DART SDK REVISION: Dart SDK version: 2.13.0-edge.5129cff930d9039852c4a037e096fd97551eec1e (be) (Tue Apr 13 02:36:16 2021 +0000) on "linux_x64"


dart runtime/tools/dartfuzz/dartfuzz.dart --no-fp --no-ffi --flat --seed 3586617624 fuzz.dart

-- RUN 1 --

DART_CONFIGURATION='DebugX64C' DART_VM_FLAGS='--enable-asserts' pkg/vm/tool/precompiler2 fuzz.dart snapshot
pkg/vm/tool/dart_precompiled_runtime2 snapshot

-- RUN 2 --

out/ReleaseSIMARM64C/dart --use-slow-path --old_gen_heap_size=128 /b/s/w/itX742Di/dart_fuzzHNTXVL/fuzz.dart

-- END REPRODUCE  --

Divergence

// AOT-DebugX64C
...
MapEntry(: false)
MapEntry(😀: 44)
MapEntry(L+- oE: #) <-------

vs

// JIT-SLOWPATH-ReleaseSIMARM64C
...
MapEntry(: false)
MapEntry(😀: 44)
MapEntry(: cX♥6(Bu) <-------
@bkonyi bkonyi added area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. dartfuzz Found with Dart fuzzing (DartFuzz, libFuzzer, etc.) labels Apr 13, 2021
dart-bot pushed a commit that referenced this issue Apr 14, 2021
@rmacnak-google
[test] Add regression test for issue 45674.
b2a1e75 
Fixed by 5564578.

TEST=ci
Bug: #45674
Change-Id: Ibe3872710b98e69c1ac9dd216c424e8b42536331
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/195187
Reviewed-by: Ben Konyi <bkonyi@google.com>
Commit-Queue: Ryan Macnak <rmacnak@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmacnak-google rmacnak-google

Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. dartfuzz Found with Dart fuzzing (DartFuzz, libFuzzer, etc.)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rmacnak-google @bkonyi