[webkit-changes] [WebKit/WebKit] 92d9b9: [JSC] Remove ValueProfile from tail calls

Yusuke Suzuki noreply at github.com
Wed Jul 26 14:13:50 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 92d9b98e0b6c8c155478fe2b6bcb326403b4dbba
      https://github.com/WebKit/WebKit/commit/92d9b98e0b6c8c155478fe2b6bcb326403b4dbba
  Author: Yusuke Suzuki <ysuzuki at apple.com>
  Date:   2023-07-26 (Wed, 26 Jul 2023)

  Changed paths:
    M JSTests/stress/arith-abs-on-various-types.js
    M JSTests/stress/arith-acos-on-various-types.js
    M JSTests/stress/arith-acosh-on-various-types.js
    M JSTests/stress/arith-asin-on-various-types.js
    M JSTests/stress/arith-asinh-on-various-types.js
    M JSTests/stress/arith-atan-on-various-types.js
    M JSTests/stress/arith-atanh-on-various-types.js
    M JSTests/stress/arith-cbrt-on-various-types.js
    M JSTests/stress/arith-ceil-on-various-types.js
    M JSTests/stress/arith-clz32-on-various-types.js
    M JSTests/stress/arith-cos-on-various-types.js
    M JSTests/stress/arith-cosh-on-various-types.js
    M JSTests/stress/arith-expm1-on-various-types.js
    M JSTests/stress/arith-floor-on-various-types.js
    M JSTests/stress/arith-fround-on-various-types.js
    M JSTests/stress/arith-log-on-various-types.js
    M JSTests/stress/arith-log10-on-various-types.js
    M JSTests/stress/arith-log2-on-various-types.js
    M JSTests/stress/arith-negate-on-various-types.js
    M JSTests/stress/arith-round-on-various-types.js
    M JSTests/stress/arith-sin-on-various-types.js
    M JSTests/stress/arith-sinh-on-various-types.js
    M JSTests/stress/arith-sqrt-on-various-types.js
    M JSTests/stress/arith-tan-on-various-types.js
    M JSTests/stress/arith-tanh-on-various-types.js
    M JSTests/stress/arith-trunc-on-various-types.js
    M Source/JavaScriptCore/bytecode/BytecodeList.rb
    M Source/JavaScriptCore/bytecode/CodeBlock.cpp
    M Source/JavaScriptCore/bytecode/CodeBlock.h
    M Source/JavaScriptCore/bytecode/Opcode.h
    M Source/JavaScriptCore/dfg/DFGGraph.cpp
    M Source/JavaScriptCore/dfg/DFGNode.h
    M Source/JavaScriptCore/jit/JITCall.cpp
    M Source/JavaScriptCore/llint/LowLevelInterpreter.asm
    M Source/JavaScriptCore/llint/LowLevelInterpreter32_64.asm
    M Source/JavaScriptCore/llint/LowLevelInterpreter64.asm

  Log Message:
  -----------
  [JSC] Remove ValueProfile from tail calls
https://bugs.webkit.org/show_bug.cgi?id=259471
rdar://112820068

Reviewed by Keith Miller.

This patch removes ValueProfile from tail calls since they are never used effectively (because of tail calls!).

* Source/JavaScriptCore/bytecode/BytecodeList.rb:
* Source/JavaScriptCore/bytecode/CodeBlock.cpp:
(JSC::CodeBlock::finishCreation):
* Source/JavaScriptCore/bytecode/Opcode.h:
* Source/JavaScriptCore/jit/JITCall.cpp:
(JSC::JIT::compileOpCall):
* Source/JavaScriptCore/llint/LowLevelInterpreter.asm:
* Source/JavaScriptCore/llint/LowLevelInterpreter32_64.asm:
* Source/JavaScriptCore/llint/LowLevelInterpreter64.asm:

Canonical link: https://commits.webkit.org/266328@main




More information about the webkit-changes mailing list