-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JSC] Vector operations should work with CSE #9979
[JSC] Vector operations should work with CSE #9979
Conversation
EWS run on previous version of this PR (hash 240c2d3) |
240c2d3
to
0ca1e20
Compare
EWS run on previous version of this PR (hash 0ca1e20) |
0ca1e20
to
7a46965
Compare
EWS run on previous version of this PR (hash 7a46965) |
7a46965
to
9846b4f
Compare
EWS run on previous version of this PR (hash 9846b4f) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me. I'm kinda trusting you that the number of children for each Vector B3 op is correct. Otherwise I hope tests would have exploded.
Added assertions for them. |
9846b4f
to
4bfb374
Compare
https://bugs.webkit.org/show_bug.cgi?id=252114 rdar://105334819 Reviewed by Keith Miller. This patch integrates Vector operations into B3 CSE by defining Value::key and ValueKey for them. We also clean up some vector related code for generating code. * Source/JavaScriptCore/b3/B3LowerToAir.cpp: * Source/JavaScriptCore/b3/B3SIMDValue.h: * Source/JavaScriptCore/b3/B3Value.cpp: (JSC::B3::Value::key const): * Source/JavaScriptCore/b3/B3Value.h: * Source/JavaScriptCore/b3/B3ValueKey.cpp: (JSC::B3::ValueKey::materialize const): * Source/JavaScriptCore/b3/B3ValueKey.h: (JSC::B3::ValueKey::simdInfo const): (JSC::B3::ValueKey::operator== const): * Source/JavaScriptCore/b3/B3ValueKeyInlines.h: (JSC::B3::ValueKey::ValueKey): * Source/JavaScriptCore/jit/SIMDInfo.h: (JSC::SIMDInfo::operator==): (JSC::SIMDInfo::operator!=): Canonical link: https://commits.webkit.org/260223@main
4bfb374
to
151366f
Compare
Committed 260223@main (151366f): https://commits.webkit.org/260223@main Reviewed commits have been landed. Closing PR #9979 and removing active labels. |
151366f
4bfb374
π wincairoπ§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-wk1π§ͺ api-gtkπ π§ͺ jscπ tvπ§ͺ mac-wk2π jsc-armv7π§ͺ mac-AS-debug-wk2π§ͺ jsc-armv7-testsπ watchπ§ͺ mac-wk2-stressπ jsc-mipsπ§ͺ jsc-mips-tests