WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
263390
Don't emit shifts in macro assembler when shift amount is zero
https://bugs.webkit.org/show_bug.cgi?id=263390
Summary
Don't emit shifts in macro assembler when shift amount is zero
David Degazio
Reported
2023-10-19 12:42:40 PDT
rdar://117220277
When generating a shift or rotation in the JSC macro assembler, we should check if the shift amount is zero, and either emit nothing or emit a move in that case. Even though we can check for this pattern during compiler optimizations, it's probably best to check in the macro assembler anyway so we still handle places where the macro assembler is used directly (baseline JITs, inline caches, various assembly helpers).
Attachments
Add attachment
proposed patch, testcase, etc.
David Degazio
Comment 1
2023-10-19 12:47:32 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/19291
EWS
Comment 2
2023-10-20 17:51:42 PDT
Committed
269608@main
(003f045ae01b): <
https://commits.webkit.org/269608@main
> Reviewed commits have been landed. Closing PR #19291 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug