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.

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

Fix hwintrinsicChild->isContained() assert #90556

Merged
merged 10 commits into from
Oct 4, 2023
Merged

Conversation

EgorBo
Copy link
Member

@EgorBo EgorBo commented Aug 14, 2023

Fixes #90508

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Aug 14, 2023
@ghost ghost assigned EgorBo Aug 14, 2023
@ghost
Copy link

ghost commented Aug 14, 2023

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

Issue Details

null

Author: EgorBo
Assignees: EgorBo
Labels:

area-CodeGen-coreclr

Milestone: -

@kunalspathak
Copy link
Member

are you still working on this @EgorBo ?

@BruceForstall
Copy link
Member

are you still working on this @EgorBo ?

If not, please make it "Draft"

@EgorBo
Copy link
Member Author

EgorBo commented Sep 18, 2023

Oops, didn't get notifications for this one -- yes, I'll finish it this week, fixes #90508

@JulieLeeMSFT JulieLeeMSFT added this to the 8.0.0 milestone Sep 20, 2023
@EgorBo EgorBo reopened this Sep 26, 2023
EgorBo and others added 2 commits September 26, 2023 20:48
@JulieLeeMSFT
Copy link
Member

@EgorBo, I reran the failed tests and it alls passed. Is it ready to merge?

@EgorBo
Copy link
Member Author

EgorBo commented Oct 2, 2023

Should be ready now, waiting for spmi diffs

@EgorBo
Copy link
Member Author

EgorBo commented Oct 3, 2023

@dotnet/jit-contrib PTAL, a fix for a containment bug (it used to "contain" some nodes such as locals while we can only contain constants and memory loads)

v + Sse3.MoveAndDuplicate(Vector128.CreateScalarUnsafe(b));

[MethodImpl(MethodImplOptions.NoInlining)]
private static Vector128<double> Test2(Vector128<double> v, double b) =>
Copy link
Member

Choose a reason for hiding this comment

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

nit: b in Test2 and Test3 are unused.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, indeed, fixed!

Copy link
Member

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

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

LGTM

@EgorBo EgorBo merged commit bb32921 into dotnet:main Oct 4, 2023
135 of 138 checks passed
@ghost ghost locked as resolved and limited conversation to collaborators Nov 3, 2023
@EgorBo
Copy link
Member Author

EgorBo commented Jan 1, 2024

/backport to release/8.0-staging

@github-actions github-actions bot unlocked this conversation Jan 1, 2024
Copy link
Contributor

github-actions bot commented Jan 1, 2024

Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/7380228196

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 1, 2024
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.

Assertion failed 'hwintrinsicChild->isContained()'
4 participants