build: enable maglev for Linux on s390x by richardlau · Pull Request #60863 · nodejs/node · GitHub
Skip to content

build: enable maglev for Linux on s390x#60863

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
richardlau:s390x-maglev
Nov 28, 2025
Merged

build: enable maglev for Linux on s390x#60863
nodejs-github-bot merged 1 commit intonodejs:mainfrom
richardlau:s390x-maglev

Conversation

@richardlau
Copy link
Copy Markdown
Member

maglev was enabled by default for Linux on s390x in upstream V8 in 13.1 by http://crrev.com/c/5899453.

Marking as semver-major for consistency with #51360.

@richardlau richardlau added semver-major PRs that contain breaking changes and should be released in the next major version. s390x Issues and PRs related to the s390x architecture. labels Nov 26, 2025
@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Nov 26, 2025
@richardlau richardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 27, 2025
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 27, 2025
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@richardlau richardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 27, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 28, 2025
@nodejs-github-bot nodejs-github-bot merged commit 3c92ee1 into nodejs:main Nov 28, 2025
80 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

JamieMagee added a commit to JamieMagee/node that referenced this pull request Apr 21, 2026
V8's Maglev compiler has supported riscv64 since V8 14.0 (with full
source files in deps/v8/src/maglev/riscv/), but Node.js never wired
it up:

- configure.py excluded riscv64 from maglev_enabled_architectures
- tools/v8_gypfiles/v8.gyp lacked GN-scraper conditions for riscv64
  Maglev sources in both the v8_internal_headers and
  v8_base_without_compiler blocks

This adds riscv64 to both, following the same pattern used for s390x
in nodejs#60863 and matching V8's own BUILD.gn which already lists riscv64
alongside arm, arm64, x64, s390x, and ppc64 as Maglev-enabled
architectures.

Refs: nodejs/build#4099
Signed-off-by: Jamie Magee <jamie.magee@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. s390x Issues and PRs related to the s390x architecture. semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants