Skip to content

Fix GJS_DISABLE_JIT not fully disabling JIT

Ivan Molodetskikh requested to merge YaLTeR/gjs:really-disable-jit into master

This seems to be the third option that needs to be set to 0 to completely disable JIT.

EnterJitStatus js::jit::MaybeEnterJit(JSContext* cx, RunState& state) {
  if (!IsBaselineInterpreterEnabled()) {
    // All JITs are disabled.
    return EnterJitStatus::NotEntered;
  }

  // ...
inline bool IsBaselineInterpreterEnabled() {
#ifdef JS_CODEGEN_NONE
  return false;
#else
  return JitOptions.baselineInterpreter && JitOptions.supportsFloatingPoint;
#endif
}

After this change EnterJit no longer shows up on my perf stacks, which is what I wanted.

Merge request reports