Skip to content

Conversation

@SingleAccretion
Copy link
Contributor

@SingleAccretion SingleAccretion commented Sep 25, 2022

One more step towards ADDR deletion.

Some small improvements and some smaller regressions from copy propagation. The regression in the tests collection is large because it is duplicated across many (generated) HWI tests.

@ghost ghost added area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI community-contribution Indicates that the PR has been added by a community member labels Sep 25, 2022
@ghost
Copy link

ghost commented Sep 25, 2022

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Issue Details

One more step towards ADDR deletion.

Some small improvements and some smaller regressions from copy propagation.

Author: SingleAccretion
Assignees: -
Labels:

area-CodeGen-coreclr

Milestone: -

@SingleAccretion SingleAccretion force-pushed the LclMorph-Primitives branch 2 times, most recently from 638353d to e9b0dee Compare September 26, 2022 12:32
Only for RETURN users for now.
We should really delete TYP_BOOL.
The code isn't actually set up for this. No diffs.
@SingleAccretion SingleAccretion marked this pull request as ready for review September 30, 2022 17:47
@SingleAccretion
Copy link
Contributor Author

@dotnet/jit-contrib

}

// See if the variable is equal to another variable or a constant.
// See if the variable is equal to another variable.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strange, I thought I had already removed this.

Also fgMorphTree already iterates, assuming there are opportunities to chain assertions.

@AndyAyersMS AndyAyersMS merged commit ca6fb31 into dotnet:main Oct 2, 2022
@AndyAyersMS
Copy link
Member

Thank you for your continued work on the jit.

@ghost ghost locked as resolved and limited conversation to collaborators Nov 11, 2022
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 community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants