Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
3c4ccbc
Change all "unmanaged" (no GC fields) sequential types to have sequen…
jkoritzinsky Nov 18, 2021
6b9e41d
Merge branch 'main' into managedseqential_unmanaged
jkoritzinsky Nov 18, 2021
b85447d
Fix inverted condition
jkoritzinsky Nov 19, 2021
0ff8540
Account for RequiresAlign8 for auto-layout nested structs in sequenti…
jkoritzinsky Nov 22, 2021
c6ee36d
Clean up R2R layout algorithm's handling of the new managed-sequentia…
jkoritzinsky Nov 29, 2021
99c0244
Update src/coreclr/tools/aot/ILCompiler.ReadyToRun/Compiler/ReadyToRu…
jkoritzinsky Nov 30, 2021
0ec572a
Try to remove the explicit layout quirk by following the behavior in …
jkoritzinsky Nov 30, 2021
e2589f3
Fix missed cast.
jkoritzinsky Nov 30, 2021
1744d82
Update src/coreclr/tools/Common/TypeSystem/Common/MetadataFieldLayout…
jkoritzinsky Nov 30, 2021
bbaa220
AlignUp the right variable.
jkoritzinsky Nov 30, 2021
7afb6d2
Remove alignUpInstanceByteSize parameter as it's always true
jkoritzinsky Nov 30, 2021
f8f8390
Merge branch 'main' of github.com:dotnet/runtime into managedseqentia…
jkoritzinsky Nov 30, 2021
0e317fe
Make enums use sequential layout again since the auto-layout algorith…
jkoritzinsky Nov 30, 2021
f174cad
Remove HasLayoutMetadata branch and method since the only cases where…
jkoritzinsky Nov 30, 2021
6a90bb3
Handle enum alignment concerns while allowing enums themselves to hav…
jkoritzinsky Dec 3, 2021
880430a
Update the auto-layout algorithm to better track alignment requiremen…
jkoritzinsky Dec 8, 2021
83e7c80
Merge branch 'main' of github.com:dotnet/runtime into managedseqentia…
jkoritzinsky Dec 9, 2021
91df6da
Remove now-invalid alignment fixup
jkoritzinsky Dec 9, 2021
8196fbe
Handle the case of an auto-layout struct as a field in another struct.
jkoritzinsky Dec 9, 2021
da12b83
Store the custom alignment requirement for layoutkind.auto value type…
jkoritzinsky Dec 10, 2021
47c95bf
Update StructPacking test based on failures.
jkoritzinsky Dec 10, 2021
c309a69
Fix more of the struct packing test results
jkoritzinsky Dec 10, 2021
6cfe0c7
Don't double-count the adjustment for auto-layout reference type base…
jkoritzinsky Dec 10, 2021
f6af360
Fix GCSeries calculation for platforms that have the specialized alig…
jkoritzinsky Dec 10, 2021
66f80e8
Update check to specifically check if the base size is pointer sized.…
jkoritzinsky Dec 10, 2021
8501319
Update StructPacking.cs for arm and arm64 (we now respect alignment r…
jkoritzinsky Dec 10, 2021
1b9eda3
Fix default packing size calculations
jkoritzinsky Dec 10, 2021
2f74ce3
Try one more change to fix the last issue
jkoritzinsky Dec 11, 2021
c98ae95
Cleanup and update some corner cases of the managed-sequential case.
jkoritzinsky Dec 13, 2021
966af36
Fix nites.
jkoritzinsky Dec 13, 2021
ae9aa6c
Fix some copy-paste issues.
jkoritzinsky Dec 13, 2021
842aef2
Fix ->/. typo
jkoritzinsky Dec 13, 2021
d269851
Fix disqualifying from managed-sequential when RequiresAlign8 is true…
jkoritzinsky Dec 13, 2021
e423c3c
Update x86 tests that check sizeof for types that have changed size.
jkoritzinsky Dec 14, 2021
a1bc86e
Fix Align8 candidate calculation to not account for packing for auto-…
jkoritzinsky Dec 14, 2021
53e5644
Merge branch 'managedseqential_unmanaged' of github.com:jkoritzinsky/…
jkoritzinsky Dec 14, 2021
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 disqualifying from managed-sequential when RequiresAlign8 is true…
… and the type has GC pointers.
  • Loading branch information
jkoritzinsky committed Dec 13, 2021
commit d2698519e758b3f3152b0f0d511e6f3e73aaedd9
5 changes: 2 additions & 3 deletions src/coreclr/vm/classlayoutinfo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -299,14 +299,13 @@ namespace
// this field type has GC pointers in it, which need to be pointer-size aligned
// so do this if it has not been done already
pManagedPlacementInfo->m_alignment = TARGET_POINTER_SIZE;
// Types that have GC Pointer fields (objects or byrefs) are disqualified from ManagedSequential layout.
return TRUE;
}
else
{
pManagedPlacementInfo->m_alignment = pNestedType.GetMethodTable()->GetFieldAlignmentRequirement();
}
return FALSE;
// Types that have GC Pointer fields (objects or byrefs) are disqualified from ManagedSequential layout.
return pNestedType.GetMethodTable()->ContainsPointers() != FALSE;
}

// No other type permitted for ManagedSequential.
Expand Down