Bug 251769

Summary:	Remove unnecessary masking after wasm LLInt comparison instructions.
Product:	WebKit	Reporter:	Mark Lam <mark.lam>
Component:	JavaScriptCore	Assignee:	Mark Lam <mark.lam>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Mark Lam

Reported 2023-02-05 17:41:25 PST

Currently, we always follow a LLInt comparison instructions with an `and with 1` mask. This is completely unnecessary because the LLInt comparison instructions already always produce a 1 or a 0. Hence, the `and with 1` mask is effectively a no-op. This patch removes this unnecessary masking.

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2023-02-05 17:41:49 PST

<rdar://problem/105063662>

Mark Lam

Comment 2 2023-02-05 17:48:29 PST

Pull request: https://github.com/WebKit/WebKit/pull/9666

EWS

Comment 3 2023-02-06 01:15:50 PST

Committed 259882@main (9354d948240f): <https://commits.webkit.org/259882@main> Reviewed commits have been landed. Closing PR #9666 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.