Fix invalid IL in RefFields test - remove ldind.ref on value type references

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[Copilot]

This PR fixes the intermittent failure of the Loader/classloader/RefFields/Validate test that was causing assertion failures in methodtable.cpp with the error:

ASSERT FAILED
Expression: (GetComponentSize() <= 2) || IsArray()
Location: line 6277 in /__w/1/s/src/coreclr/vm/methodtable.cpp
Function: SanityCheck

Root Cause

The issue was in the WithRefStructField.ConfirmFieldInstance method in InvalidCSharp.il, which contained invalid IL:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldind.ref// ❌ INVALID: ldind.ref on value type referenceldarg.1ldind.ref// ❌ INVALID: ldind.ref on value type reference ceqret
}

The problem was using ldind.ref to dereference WithRefField& (reference to a value type). ldind.ref is only valid for dereferencing object references, not value type references. This caused the JIT to generate incorrect GC information, leading to the assertion failure.

Fix

The fix removes the invalid ldind.ref instructions and directly compares the two WithRefField& references:

.methodpublichidebysiginstancebool ConfirmFieldInstance (
valuetype InvalidCSharp.WithRefField&
) cilmanaged
{
ldarg.0ldfldvaluetype InvalidCSharp.WithRefField& InvalidCSharp.WithRefStructField::Field
ldarg.1ceqret
}

This correctly compares whether the stored reference in the field points to the same location as the parameter reference, which is the intended behavior of the test.

Verification

• Verified the IL assembles correctly using ilasm
• Confirmed the syntax follows proper IL patterns for reference comparison
• The fix addresses the root cause of the intermittent GC-related crashes

Fixes#116358.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• k0ivsblobprodcus356.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)
• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/Validate.csproj NuGetInteractive=false;MSBuildRestoreSessionId=21e5ac85-7729-4227-9df0-d7f3c485866d;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[AaronRobinsonMSFT]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref are identical. However, the approach I took was incorrect for value types. Instead, the solution should be to simply compare the managed pointers instead of dereferencing them in IL. The easiest way to address this would be to remove both ldind.ref instructions in the method mentioned in the referenced comment above.

[Copilot]

@copilot The issue here is captured in the comment by @jkotas, #116358 (comment). The point of this test was to validate the targets of the byref ...

Done. Both ldind.ref instructions have been removed from the WithRefStructField.ConfirmFieldInstance method. The method now directly compares the managed pointers instead of dereferencing them, which is the correct approach for value type references. The IL assembles successfully without errors. (bb97cb7)

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• mfjvsblobprodcus373.vsblob.vsassets.io
  • Triggering command: /home/REDACTED/work/runtime/runtime/.dotnet/dotnet /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/NuGet.Build.Tasks.Console.dll Recursive=True;CleanupAssetsForUnsupportedProjects=True;DisableParallel=False;Force=False;ForceEvaluate=False;HideWarningsAndErrors=False;IgnoreFailedSources=False;Interactive=False;NoCache=False;NoHttpCache=False;RestorePackagesConfig=False /home/REDACTED/work/runtime/runtime/.dotnet/sdk/10.0.100-preview.6.25272.112/MSBuild.dll /home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields/InvalidCSharp.ilproj NuGetInteractive=false;MSBuildRestoreSessionId=4becf198-a8ff-4047-b763-206cb6de1a72;MSBuildIsRestoring=True;ExcludeRestorePackageImports=True;OriginalMSBuildStartupDirectory=/home/REDACTED/work/runtime/runtime/src/tests/Loader/classloader/RefFields (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list