Skip to content

Disable Int128 use in by value ABI scenarios, and fix field layout behavior #74123

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
davidwrighton merged 12 commits into from
Aug 19, 2022
Merged

Disable Int128 use in by value ABI scenarios, and fix field layout behavior #74123

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
29ea888
First stab at support for proper 128bit integer layout and abi
davidwrighton Aug 17, 2022
ca40e03
Add ABI tests for Int128 covering interesting scenarios
davidwrighton Aug 9, 2022
183a36e
Fix bugs so that at least Windows Arm64 works
davidwrighton Aug 17, 2022
823090b
Add more types to the ABI tester, so that we cover the Int128 scenarios
davidwrighton Aug 18, 2022
22adce8
Revert changes which attempted to enable by value passing for Int128
davidwrighton Aug 18, 2022
e36292f
Make Int128 have layout match the expected unmanaged field layout
davidwrighton Aug 18, 2022
867ab2e
Mark Int128 types as not having a stable abi
davidwrighton Aug 18, 2022
8585002
Address all known issues
davidwrighton Aug 19, 2022
fdba562
Merge branch 'main' of github.com:dotnet/runtime into fix_72206_2
davidwrighton Aug 19, 2022
4fab4d3
Try to fix PR job
davidwrighton Aug 19, 2022
1170afe
Should fix the test issues
davidwrighton Aug 19, 2022
81c4439
Apply suggestions from code review
davidwrighton Aug 19, 2022
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
Apply suggestions from code review 
Co-authored-by: Jeremy Koritzinsky <[email protected]>
  • Loading branch information
@davidwrighton @jkoritzinsky
davidwrighton and jkoritzinsky authored Aug 19, 2022
commit 81c4439e3a33dc32b5dbd4f81222cbb779fd1407
4 changes: 0 additions & 4 deletions src/coreclr/vm/classlayoutinfo.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -330,10 +330,6 @@ namespace

BOOL TypeHasInt128Field(CorElementType corElemType, TypeHandle pNestedType)
{
if (CorTypeInfo::IsPrimitiveType(corElemType) || corElemType == ELEMENT_TYPE_PTR || corElemType == ELEMENT_TYPE_FNPTR)
{
return FALSE;
}
if (corElemType == ELEMENT_TYPE_VALUETYPE)
{
_ASSERTE(!pNestedType.IsNull());
Expand Down