tree a6ef75b848489f95dcbbeb126888fe2c6a33ab5f
parent 33feee3b030c24f85a7f5a5c8475ef4f7aa1780b
author Aart Bik <ajcbik@google.com> 1528931704 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> 1528931704 +0000

[vm/inliner] Add smi return type of clamped convertors.

Rationale:
Rather than forced inlining of clamped convertors
(the "saturated" method _toClampedUint8() from
dart:typed_data), exposing the fact that it always
returns smi values (since they check fail on null inputs)
yields much better Uint8ClampedListView performance
(it avoids re-compilation due to a speculative CheckSmi).

Note:
In the long run, we may still want them inlined
and improve range analysis to deal with clamping.

Performance:
About 2.8x faster than previous optimized version,
about 4.5x faster than original.

https://github.com/dart-lang/sdk/issues/33205

Change-Id: I86a06525d2f2ea0476effd3c3d856ff8d9ab1d87
Reviewed-on: https://dart-review.googlesource.com/60201
Commit-Queue: Aart Bik <ajcbik@google.com>
Reviewed-by: Alexander Markov <alexmarkov@google.com>