1996605 - Pass CycleCollectedJSContext to CallSetup constructors where available

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Matthew Gaudet (he/him) [:mgaudet]]

Currently there's are two TLS lookups happening during CallSetup for microtask handling, but these aren't actually necessary as we could pipe the CycleCollectedJSContext through avoiding them in the common case.

See this profile of Doxbee-promise showing TLS lookup as the top self-function: https://share.firefox.dev/4ozDHjf

Comment 1

[Bryan Thrall [:bthrall]]

Attachment: Bug 1996605 - Get CycleCollectedJSContext once instead of twice in CallSetup r=mgaudet!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/bf30ffb3cfae
https://hg.mozilla.org/integration/autoland/rev/c37ed92d59b0
Get CycleCollectedJSContext once instead of twice in CallSetup r=smaug

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/c37ed92d59b0