Skip to content

[JSC] testair: remove redundant #if#61516

Merged
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
dhecht:eng/JSC-testair-remove-redundant-if
Mar 28, 2026
Merged

[JSC] testair: remove redundant #if#61516
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
dhecht:eng/JSC-testair-remove-redundant-if

Conversation

@dhecht
Copy link
Copy Markdown
Contributor

@dhecht dhecht commented Mar 27, 2026

@dhecht dhecht requested a review from a team as a code owner March 27, 2026 19:17
@dhecht dhecht self-assigned this Mar 27, 2026
@dhecht dhecht added the JavaScriptCore For bugs in JavaScriptCore, the JS engine used by WebKit, other than kxmlcore issues. label Mar 27, 2026
@dhecht dhecht added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 28, 2026
https://bugs.webkit.org/show_bug.cgi?id=310922
rdar://173527143

Reviewed by Yijia Huang.

This #if USE(JSVALUE64) is already enclosed in an #if USE(JSVALUE64)
block starting at line 2464, so remove it.

Test: Source/JavaScriptCore/b3/air/testair.cpp
* Source/JavaScriptCore/b3/air/testair.cpp:

Canonical link: https://commits.webkit.org/310128@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/JSC-testair-remove-redundant-if branch from 3545fa2 to fccbc62 Compare March 28, 2026 01:37
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 310128@main (fccbc62): https://commits.webkit.org/310128@main

Reviewed commits have been landed. Closing PR #61516 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit fccbc62 into WebKit:main Mar 28, 2026
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

JavaScriptCore For bugs in JavaScriptCore, the JS engine used by WebKit, other than kxmlcore issues.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants