Skip to content

Conversation

@CarolEidt
Copy link
Contributor

  • Fix an incorrect short-circuit for bestPossibleScore
  • Don't set delayRegFree on an operand that is the same as the one in the RMW position, and one of them is a last use.
  • Add REG_LR to the REG_VAR_ORDER

These changes have minor impact, but make it easier to achieve zero diffs when changing how the heuristics are computed.

@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Oct 6, 2020
- Fix an incorrect short-circuit for bestPossibleScore
- Don't set delayRegFree on an operand that is the same as the one in the RMW position, and one of them is a last use.
- Add REG_LR to the REG_VAR_ORDER
- Other misc. changes.
@ViktorHofer
Copy link
Member

// Auto-generated message

69e114c which was merged 12/7 removed the intermediate src/coreclr/src/ folder. This PR needs to be updated as it touches files in that directory which causes conflicts.

To update your commits you can use this bash script: https://gist.github.com/ViktorHofer/6d24f62abdcddb518b4966ead5ef3783. Feel free to use the comment section of the gist to improve the script for others.

@kunalspathak kunalspathak self-assigned this Jan 5, 2021
@kunalspathak
Copy link
Contributor

I verified the changes in this PR and all of them were ported and merged in #45135.

@ghost ghost locked as resolved and limited conversation to collaborators Feb 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants