Skip to content

Fix nested non-byref like VC with no pointer fields #80642

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

Merged
AaronRobinsonMSFT merged 6 commits into from
Jan 19, 2023
Merged

Fix nested non-byref like VC with no pointer fields #80642

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b69e03e
Fix nested non-byref like VC with no pointer fields
AaronRobinsonMSFT Jan 14, 2023
21bf6f5
Add tests
AaronRobinsonMSFT Jan 14, 2023
0687977
Fix test typo
AaronRobinsonMSFT Jan 14, 2023
0577484
Disable new test on llvmaot
AaronRobinsonMSFT Jan 15, 2023
09365ed
Apply suggestions from code review
AaronRobinsonMSFT Jan 15, 2023
142862a
Fully enable new test for MonoAOT
AaronRobinsonMSFT Jan 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix test typo
  • Loading branch information
@AaronRobinsonMSFT
AaronRobinsonMSFT committed Jan 14, 2023
commit 068797743fc6ca4f9d0d1489cbc9a61815f9625e
4 changes: 2 additions & 2 deletions src/tests/Loader/classloader/explicitlayout/objrefandnonobjrefoverlap/case13.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ public struct WithNoORefs
public int F;
}

public ref struct WithByORefs
public ref struct WithByRefs
{
public ref int F;
}
Expand Down Expand Up @@ -50,7 +50,7 @@ public ref struct Explicit3
[FieldOffset(0)] public Inner3 Field1;
public ref struct Inner3
{
public WithByORefs Field2;
public WithByRefs Field2;
}
}

Expand Down