-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[S390X] remove redundant load #119840
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
[S390X] remove redundant load #119840
Conversation
Remove extra redudant load. ins->dreg is always fpr and sreg is always r13 so they can't be aliased as well, so it's a redundant load.
|
FYI: Please do not merge the fix as we would like to test our independent CI bot for s390x :)) |
|
@Dotnet-s390x build |
|
Build Queued.. To cancel the current build, please comment: |
|
Build starting on s390x... |
|
** Build Failed ** Build Error Summary |
|
@Dotnet-s390x build |
|
Build Queued.. To cancel the current build, please comment: |
|
@Dotnet-s390x cancel |
|
Build CANCELLED. |
|
@Dotnet-s390x dummy-flag |
Dotnet-s390x Bot InstructionsTo start a .NET runtime build on s390x, comment: To cancel a running build: |
|
@Dotnet-s390x build |
|
Build Queued.. To cancel the current build, please comment: |
|
Build Failed Build Error Summary |
|
@Dotnet-s390x build |
|
Build Queued.. To cancel the current build, please comment: |
|
Build Successful |
@steveisok @vitek-karas looks like the build is successful with no test failures, I think this can be merged now. |
|
@saitama951 sorry looks like this fell through the cracks, is this still needed? |
|
@akoeplinger yes, this removes an extra redundant load for the specific IR. |
|
/ba-g browser-wasm failures can't be related |
Remove extra redudant load.
ins->dreg is always fpr and sreg is always r13 so they can't be aliased as well, so it's a redundant load.