[webkit-changes] [WebKit/WebKit] 71f803: [JSC] Add aligned label annotation to offlineasm

Commit Queue noreply at github.com
Thu Mar 14 06:01:43 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 71f803e6ee62ac5bcf8449a8e9cb1d4fd601f74a
      https://github.com/WebKit/WebKit/commit/71f803e6ee62ac5bcf8449a8e9cb1d4fd601f74a
  Author: Max Rottenkolber <maximilian at igalia.com>
  Date:   2024-03-14 (Thu, 14 Mar 2024)

  Changed paths:
    M Source/JavaScriptCore/llint/InPlaceInterpreter.asm
    M Source/JavaScriptCore/llint/LowLevelInterpreter.cpp
    M Source/JavaScriptCore/offlineasm/asm.rb
    M Source/JavaScriptCore/offlineasm/ast.rb
    M Source/JavaScriptCore/offlineasm/backends.rb
    M Source/JavaScriptCore/offlineasm/parser.rb
    M Source/JavaScriptCore/offlineasm/transform.rb

  Log Message:
  -----------
  [JSC] Add aligned label annotation to offlineasm
https://bugs.webkit.org/show_bug.cgi?id=270699

Reviewed by Yusuke Suzuki and Justin Michaud.

To express some programs like the InPlaceInterpreter we need to align
instructions to specific boundaries (such as 256).

We currently to do this via `emit ".balign 256, 0xcc"' (for instance)
but this is not reliable on armv7 as its globaladdr implementation can
insert code unpredictably (from the point of view of the offlineasm
user), hence breaking the alignment.

This adds an aligned label annotation which implements this
functionality in offlineasm.

* Source/JavaScriptCore/llint/InPlaceInterpreter.asm:
* Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:
* Source/JavaScriptCore/offlineasm/asm.rb:
* Source/JavaScriptCore/offlineasm/ast.rb:
* Source/JavaScriptCore/offlineasm/backends.rb:
* Source/JavaScriptCore/offlineasm/parser.rb:
* Source/JavaScriptCore/offlineasm/transform.rb:

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list