Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ARM64: Redundant movs done for zero extend the register #35254

Closed
kunalspathak opened this issue Apr 21, 2020 · 0 comments · Fixed by #62630
Closed

ARM64: Redundant movs done for zero extend the register #35254

kunalspathak opened this issue Apr 21, 2020 · 0 comments · Fixed by #62630
Assignees
@EgorBo
Labels
arch-arm64 area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI optimization
Milestone
Future

Comments

@kunalspathak
Copy link
Member

kunalspathak commented Apr 21, 2020 •
edited by BruceForstall
Loading

Redundant movs are done to zero the upper 32-bit of register, but if they are done after ldr, we can eliminate them.

B8736822          ldr     w2, [x1, x19]
2A0203E2          mov     w2, w2   ; can be eliminated

In framework libraries code, found 1098 such instructions that can eliminated.

Details :
redundant-mov-3.txt

category:cq
theme:optimization
skill-level:intermediate
cost:small
@kunalspathak kunalspathak added arch-arm64 area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI labels Apr 21, 2020
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added the untriaged New issue has not been triaged by the area owner label Apr 21, 2020
@BruceForstall BruceForstall added optimization and removed untriaged New issue has not been triaged by the area owner labels Apr 21, 2020
@BruceForstall BruceForstall added this to the Future milestone Apr 21, 2020
@kunalspathak kunalspathak mentioned this issue May 5, 2020
Closed
@BruceForstall BruceForstall added the JitUntriaged CLR JIT issues needing additional triage label Oct 28, 2020
@kunalspathak kunalspathak mentioned this issue Nov 9, 2020
Closed
29 tasks
@kunalspathak kunalspathak modified the milestones: Future, 6.0.0 Nov 9, 2020
@kunalspathak kunalspathak removed the JitUntriaged CLR JIT issues needing additional triage label Nov 9, 2020
@JulieLeeMSFT JulieLeeMSFT modified the milestones: 6.0.0, Future Mar 23, 2021
@echesakov echesakov mentioned this issue Jul 8, 2021
Closed
8 tasks
@EgorBo EgorBo mentioned this issue Dec 10, 2021
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Dec 10, 2021
@EgorBo EgorBo self-assigned this Dec 10, 2021
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jan 19, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Feb 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@EgorBo EgorBo

Labels
arch-arm64 area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI optimization
Projects
.NET Core CodeGen
Archived in project
Milestone
Future
Development

Successfully merging a pull request may close this issue.

[arm64] JIT: Redundant zero/sign extensions after ldrX/ldrsX EgorBo/runtime-1
5 participants
@EgorBo @BruceForstall @kunalspathak @Dotnet-GitSync-Bot @JulieLeeMSFT