Skip to content

Dart VM - bad benchmark results #7347

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.

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

Closed
DartBot opened this issue Dec 12, 2012 · 2 comments
Closed

Dart VM - bad benchmark results #7347

DartBot opened this issue Dec 12, 2012 · 2 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report

Comments

@DartBot
Copy link

DartBot commented Dec 12, 2012

This issue was originally filed by @bp74


What steps will reproduce the problem?

There is news on the web saying "The Dart VM is now 50% faster than V8 on the two Octane benchmarks". That's great and we are looking in a bright future with Dart!

Unfortunately i have Dart code which is realy slow, i have attached the benchmark and below are some numbers. The benchmark uses the "benchmark_harness" library, so the benchmark should be fine.

What is the expected output? What do you see instead?

The benchmark is creating a CanvasElement and performs a color matrix filter (grayscale filter). I checked the performance of the "getImageData" and "putImageData" methods of the CanvasRenderingContext2D which is fine, the problem are the loops and the index operations on the List.

Here are the benchmarl results:

Dartium (Dart), 22 iterations
CanvasBenchmark(RunTime): 1050500.0 us.

Chrome (JS), 1854 iterations
CanvasBenchmark(RunTime): 11316.384180790961 us.

IE9 (JS), 235 iterations
PROTOKOLL: CanvasBenchmark(RunTime): 90304.34782608696 us.

Firefox (JS), 548 iterations
CanvasBenchmark(RunTime): 38615.38461538462 us.

What version of the product are you using? On what operating system?

Dart Editor version 0.1.2_r16052
Dart SDK version 0.1.2.0_r16045

Please provide any additional information below.


Attachment:
canvas_benchmark.zip (40.64 KB)

@ghost
Copy link

ghost commented Dec 12, 2012

Set owner to @sgmitrovic.
Added Area-VM, Accepted labels.

@ghost
Copy link

ghost commented Dec 12, 2012

imageData.data returns Uint8ClampedArray, which is not inlined yet. In addition, Dartium needs to switch using VMs Uint8ClampedArray implementation.


Added Duplicate label.
Marked as being merged into #5661.

@DartBot DartBot added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report labels Dec 12, 2012
@DartBot DartBot assigned ghost Dec 12, 2012
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report
Projects
None yet
Development

No branches or pull requests

1 participant