commit db129b65259ff692a2675863a08f83f686c24029
author Benedikt Meurer <bmeurer@chromium.org> Thu Jan 18 06:16:42 2018
committer Commit Bot <commit-bot@chromium.org> Thu Jan 18 08:27:35 2018
tree 26f8c006fd5ce9e43d2b75353c1f4584ac8a1d56
parent f3d22635d15d3fff40529b00bf6ba979776ad871

[turbofan] Speculate on bounds checks for String#charAt and String#charCodeAt.

With the new builtin optimization guard we can just speculatively assume
that the index passed to String#charAt and String#charCodeAt (in optimized
code) is going to be within the valid range for the receiver. This is
what Crankshaft used to do, and it avoids Smi checks on the result for
String#charCodeAt, since it can no longer return NaN.

This gives rise to further optimizations of these builtins (i.e. to
completely avoid the tagging of char codes), and by itself already
improves the regression test originally reported from 650ms to
610ms.

Bug: v8:7127, v8:7326
Change-Id: Ia25a555c5c1a48d229c094b1ecd2487eec81e390
Reviewed-on: https://chromium-review.googlesource.com/872850
Reviewed-by: Yang Guo <yangguo@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#50667}