1816609 - JS demo (https://www.fxhash.xyz/generative/slug/vagafilium) takes longer to load in Nightly compared to Chrome

Status: NEW

(Core :: JavaScript Engine, defect, P3)

Description

[Mayank Bansal]

Go to https://www.fxhash.xyz/generative/slug/vagafilium (exact hash used is https://gateway.fxhash2.xyz/ipfs/QmRtqfsMzinLpXU6r9X9vuqTmBuHabkUuFrq5v8No8EUXb/?fxhash=ooZDukYsSwndCAQMRb5rkcs1X9VaRwnrMSJwfCSmr2VcoF9wwuS)

Nightly: https://share.firefox.dev/3jOvVGR (23s)
Chrome: https://share.firefox.dev/3YMjSID (8s)

Comment 1

[Jan de Mooij [:jandem]]

The flame graph shows we're spending most time under IonGetPropertyIC::update converting a double (outside int32 range or with a fractional part) to a PropertyKey. It might be possible to optimize/cache this better, depending on what doubles it's passing, but it's also not the best JS code.

Comment 2

[Mayank Bansal]

Latest Profile: https://share.firefox.dev/3QsWv5f (16s)

Comment 3

[Mayank Bansal]

Latest profile: https://share.firefox.dev/40uYvyj (13s)
The profile is quite different from the previous ones... there is a lot of canvas2d stuff which wasnt there in comment 2. Though it was in comment 0.

Comment 4

[Mayank Bansal]

latest profile: https://share.firefox.dev/40BEJ56 (13s)
No improvement from bug 1932864 as expected.
Bug 1941947 sounds like it would improve this. Adding dependency.

Comment 5

[Mayank Bansal]

Latest profile after Bug 1941947 is fixed: https://share.firefox.dev/40sNai3 (12.3s) .