1840350 - Remove unnecessary cx->isHelperThreadContext() check around stencil instantiation

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Tooru Fujisawa [:arai]]

We shouldn't be performing stencil instantiation off main thread now.
there's obsolete check for cx->isHelperThreadContext(), and maybe some more.

https://searchfox.org/mozilla-central/rev/d31e56f7b3c2c18b8071a7b2a2fb6b4e01e3d3e8/js/src/frontend/BytecodeCompiler.cpp#433-435,447

bool frontend::InstantiateStencils(JSContext* cx, CompilationInput& input,
                                   const CompilationStencil& stencil,
                                   CompilationGCOutput& gcOutput) {
...
  if (!cx->isHelperThreadContext()) {

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Bug 1840350 - Remove unnecessary cx->isHelperThreadContext() check around stencil instantiation. r?nbp!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/db45d5ce5f2c
Remove unnecessary cx->isHelperThreadContext() check around stencil instantiation. r=nbp

Comment 3

[Sandor Molnar[:smolnar]]

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