1881487 - Update unicode-bidi to use the smallvec optimization

Status: RESOLVED FIXED

(Core :: Internationalization, enhancement)

Description

[Jonathan Kew [:jfkthame]]

This substantially reduces heap allocations during bidi resolution, and should result in a small but measureable perf improvement.

Comment 1

[Jonathan Kew [:jfkthame]]

(See https://github.com/servo/unicode-bidi/pull/127 for the upstream optimization that we're incorporating here.)

Comment 2

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1881487 - Update unicode-bidi to pick up newly-added smallvec support. r=#platform-i18n-reviewers

Comment 3

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1881487 - Run './mach vendor rust' to get the unicode-bidi and smallvec updates. r=#platform-i18n-reviewers

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/771b0739d524
Update unicode-bidi to pick up newly-added smallvec support. r=platform-i18n-reviewers,supply-chain-reviewers,nordzilla
https://hg.mozilla.org/integration/autoland/rev/561af3572275
Run './mach vendor rust' to get the unicode-bidi and smallvec updates. r=platform-i18n-reviewers,nordzilla

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/771b0739d524
https://hg.mozilla.org/mozilla-central/rev/561af3572275

Comment 6

[Nicolas Chevobbe [:nchevobbe]]

== Change summary for alert #41572 (as of Sat, 24 Feb 2024 19:57:15 GMT) ==

Improvements:

Ratio	Test	Platform	Options	Absolute values (old vs new)
5%	damp custom.jsdebugger.open-large-minified-file.DAMP	linux1804-64-shippable-qr	e10s fission stylo webrender	1,206.71 -> 1,145.13

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=41572