-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
JIT: ARM64 SVE format encodings, SVE_BI_2A
to SVE_HF_2A
#98784
Conversation
@dotnet/jit-contrib @dotnet/arm64-contrib @kunalspathak @a74nh @amanasifkhalid this is ready. |
Just noticed that the JIT outputs |
I believe it is; the docs say so. |
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.
LGTM, will approve once the dup
s are displayed as mov
. Thanks!
Diff results for #98784Throughput diffsThroughput diffs for linux/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Throughput diffs for osx/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Throughput diffs for windows/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Details here |
I agree with this :) |
@amanasifkhalid this is ready again |
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.
LGTM, thanks!
Diff results for #98784Assembly diffsAssembly diffs for linux/arm64 ran on windows/x64Diffs are based on 2,554,585 contexts (1,019,526 MinOpts, 1,535,059 FullOpts). MISSED contexts: 172 (0.01%) Overall (+0 bytes)
FullOpts (+0 bytes)
Assembly diffs for linux/x64 ran on windows/x64Diffs are based on 2,543,224 contexts (988,245 MinOpts, 1,554,979 FullOpts). MISSED contexts: 177 (0.01%) Overall (-3 bytes)
FullOpts (-3 bytes)
Assembly diffs for osx/arm64 ran on windows/x64Diffs are based on 2,317,543 contexts (945,402 MinOpts, 1,372,141 FullOpts). MISSED contexts: 170 (0.01%) Overall (+12 bytes)
FullOpts (+12 bytes)
Assembly diffs for windows/arm64 ran on windows/x64Diffs are based on 2,402,908 contexts (955,693 MinOpts, 1,447,215 FullOpts). MISSED contexts: 174 (0.01%) Overall (+0 bytes)
FullOpts (+0 bytes)
Details here Throughput diffsThroughput diffs for linux/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Throughput diffs for osx/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Details here |
Diff results for #98784Assembly diffsAssembly diffs for linux/arm64 ran on windows/x64Diffs are based on 2,554,585 contexts (1,019,526 MinOpts, 1,535,059 FullOpts). MISSED contexts: 172 (0.01%) Overall (+0 bytes)
FullOpts (+0 bytes)
Assembly diffs for linux/x64 ran on windows/x64Diffs are based on 2,543,224 contexts (988,245 MinOpts, 1,554,979 FullOpts). MISSED contexts: 177 (0.01%) Overall (-3 bytes)
FullOpts (-3 bytes)
Assembly diffs for osx/arm64 ran on windows/x64Diffs are based on 2,317,543 contexts (945,402 MinOpts, 1,372,141 FullOpts). MISSED contexts: 170 (0.01%) Overall (+12 bytes)
FullOpts (+12 bytes)
Assembly diffs for windows/arm64 ran on windows/x64Diffs are based on 2,402,908 contexts (955,693 MinOpts, 1,447,215 FullOpts). MISSED contexts: 174 (0.01%) Overall (+0 bytes)
FullOpts (+0 bytes)
Details here Throughput diffsThroughput diffs for linux/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Throughput diffs for osx/arm64 ran on windows/x64MinOpts (-0.01% to +0.00%)
Details here |
Contributes to #94549
Adds formats:
Note: Format
SVE_HH_2A
was not able to be decoded, but the format is simple enough that I'm quite confident in it.Left: Capstone,
Right: Jit