-
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
[arm64] JIT: Add with sign/zero extend #61549
Conversation
Tagging subscribers to this area: @JulieLeeMSFT Issue Detailsbenchmarks.run.windows.arm64.checked.mch:
Detail diffs
coreclr_tests.pmi.windows.arm64.checked.mch:
Detail diffs
libraries.crossgen2.windows.arm64.checked.mch:
Detail diffs
libraries.pmi.windows.arm64.checked.mch:
Detail diffs
libraries_tests.pmi.windows.arm64.checked.mch:
Detail diffs
|
/azp run Fuzzlyn, runtime-coreclr outerloop, runtime-coreclr gcstress0x3-gcstress0xc |
Azure Pipelines successfully started running 3 pipeline(s). |
@jakobbotsch can you please take a look at the Fuzzlyn-arm64 failure, is it worth investigating?
|
I've seen it a few times, I don't think it's related to this PR. Seems like either Fuzzlyn can generate extremely large structures or something else is wrong. It's probably the former. |
@dotnet/jit-contrib PTAL, failures are not related |
@dotnet/jit-contrib ping |
This PR is supposed to fold
ADD(LSH, X)
andADD(CAST, X)
to a singleadd
instruction with uxtw/sxtw/lsl.diff example: https://www.diffchecker.com/tbiW1wP4
benchmarks.run.windows.arm64.checked.mch:
Detail diffs
coreclr_tests.pmi.windows.arm64.checked.mch:
Detail diffs
libraries.crossgen2.windows.arm64.checked.mch:
Detail diffs
libraries.pmi.windows.arm64.checked.mch:
Detail diffs
libraries_tests.pmi.windows.arm64.checked.mch:
Detail diffs