commit 734c1456d942a03d79aab4b3b0e57afbc803ceea
author Seth Brenith <seth.brenith@microsoft.com> Tue May 28 15:03:18 2019
committer Commit Bot <commit-bot@chromium.org> Tue May 28 15:51:17 2019
tree 37985ec7297b160bee93c602b915db88f2c37fd4
parent e5dcf29ab13c163f354a2293a91d2e6cde9536fd

[intl] Avoid a temporary allocation while converting to ICU string

If we need to build an icu::UnicodeString for a string that is currently
in one-byte representation, we first have to expand the string's content
into a two-byte representation. Doing so involves allocating an array,
which is slow. With this change, we can convert short strings on the
stack instead to save time. The cutoff length for what counts as "short"
is pretty arbitrary, but we believe many strings fit into an 80-column
line. This increases the score of cdjs in JetStream 2 by 35% on my
machine, because cdjs is basically a test of localeCompare throughput.

Bug: v8:9305

Change-Id: Iba081ac5a8fa7659edf06ac97ba8acf3f8328d59
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1630848
Commit-Queue: Seth Brenith <seth.brenith@microsoft.com>
Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#61901}