Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,9 @@
<Compile Include="$(CompilerCommonPath)\TypeSystem\Common\CastingHelper.cs">
<Link>Internal\TypeSystem\CastingHelper.cs</Link>
</Compile>
<Compile Include="$(CompilerCommonPath)\TypeSystem\Canon\CastingHelper.Canon.cs">
Comment thread
jkotas marked this conversation as resolved.
<Link>Internal\TypeSystem\CastingHelper.Canon.cs</Link>
</Compile>
<Compile Include="$(CompilerCommonPath)\TypeSystem\Common\DefType.cs">
<Link>Internal\TypeSystem\DefType.cs</Link>
</Compile>
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,93 @@
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.

namespace Internal.TypeSystem
{
public static partial class CastingHelper
{
/// <summary>
/// Check if <paramref name="otherType"/> is a canonical type that <paramref name="thisType"/>
/// can be cast to. __Canon accepts any reference type; __UniversalCanon accepts any type.
/// Pointers, byrefs, and function pointers are not valid instantiation arguments.
/// </summary>
private static bool IsCanonicalCastTarget(TypeDesc thisType, TypeDesc otherType)
{
TypeSystemContext context = thisType.Context;

if (context.IsCanonicalDefinitionType(otherType, CanonicalFormKind.Universal))
return true;

if (context.IsCanonicalDefinitionType(otherType, CanonicalFormKind.Specific))
return thisType.IsGCPointer;

return false;
}

/// <summary>
/// Check if two type arguments can be considered matching because one (or both) is canonical.
/// __Canon matches any reference type; __UniversalCanon matches any type.
/// </summary>
private static bool IsCanonicalTypeArgMatch(TypeDesc type, TypeDesc otherType)
{
TypeSystemContext context = type.Context;

if (context.IsCanonicalDefinitionType(otherType, CanonicalFormKind.Universal))
return true;

if (context.IsCanonicalDefinitionType(otherType, CanonicalFormKind.Specific))
return type.IsGCPointer || context.IsCanonicalDefinitionType(type, CanonicalFormKind.Any);

if (context.IsCanonicalDefinitionType(type, CanonicalFormKind.Universal))
return true;

if (context.IsCanonicalDefinitionType(type, CanonicalFormKind.Specific))
return otherType.IsGCPointer || context.IsCanonicalDefinitionType(otherType, CanonicalFormKind.Any);

// For non-leaf types (e.g., Arg2<string> vs Arg2<__Canon>), check if they are
// canon-equivalent: same type definition with canon-compatible type arguments.
if (IsCanonEquivalent(type, otherType))
return true;

// For parameterized types like arrays (string[] vs __Canon[]),
// recursively match the element/parameter type.
if (type is ParameterizedType paramType && otherType is ParameterizedType otherParamType
&& type.Category == otherType.Category)
{
if (type is ArrayType arrayType && otherType is ArrayType otherArrayType
&& arrayType.Rank != otherArrayType.Rank)
return false;

return IsCanonicalTypeArgMatch(paramType.ParameterType, otherParamType.ParameterType);
}

return false;
}

/// <summary>
/// Check if two types are equivalent considering canonical type matching rules.
/// Same type definition with all type arguments either equal or canon-compatible.
/// </summary>
private static bool IsCanonEquivalent(TypeDesc thisType, TypeDesc otherType)
{
if (!thisType.HasSameTypeDefinition(otherType))
return false;

Instantiation thisInst = thisType.Instantiation;
Instantiation otherInst = otherType.Instantiation;

if (thisInst.Length == 0)
return false;

for (int i = 0; i < thisInst.Length; i++)
{
if (thisInst[i] == otherInst[i])
continue;

if (!IsCanonicalTypeArgMatch(thisInst[i], otherInst[i]))
return false;
}

return true;
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.

using System.Diagnostics;

namespace Internal.TypeSystem
{
public static partial class TypeSystemConstraintsHelpers
{
private static bool IsSpecialTypeMeetingConstraint(TypeDesc type, GenericConstraints constraint)
{
TypeSystemContext context = type.Context;

return constraint switch
{
GenericConstraints.ReferenceTypeConstraint => context.IsCanonicalDefinitionType(type, CanonicalFormKind.Any),
GenericConstraints.DefaultConstructorConstraint => context.IsCanonicalDefinitionType(type, CanonicalFormKind.Any),
GenericConstraints.NotNullableValueTypeConstraint => context.IsCanonicalDefinitionType(type, CanonicalFormKind.Universal),
_ => throw new UnreachableException()
};
}

/// <summary>
/// Checks if <paramref name="instantiationParam"/> can satisfy the type constraint
/// <paramref name="instantiatedConstraintType"/> when the param or constraint IS a canonical
/// definition type (__Canon or __UniversalCanon). Handles wildcard semantics only;
/// structural matching (interface walking, base chain, variance) is in CastingHelper.
/// </summary>
private static bool CanCastToConstraintWithCanon(TypeDesc instantiationParam, TypeDesc instantiatedConstraintType)
{
TypeSystemContext context = instantiationParam.Context;

// If the instantiation param is a canonical definition type (__Canon or __UniversalCanon),
// it acts as a wildcard — any concrete type substituted at runtime will be validated then.
if (context.IsCanonicalDefinitionType(instantiationParam, CanonicalFormKind.Any))
return true;

// If the constraint type itself is a canonical definition type, check compatibility directly.
// E.g., "where T : U" with U=__Canon means T must be a plausible match for __Canon.
if (context.IsCanonicalDefinitionType(instantiatedConstraintType, CanonicalFormKind.Universal))
return true;
if (context.IsCanonicalDefinitionType(instantiatedConstraintType, CanonicalFormKind.Specific))
return instantiationParam.IsGCPointer;

return false;
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.

namespace Internal.TypeSystem
{
public static partial class CastingHelper
{
private static bool IsCanonicalCastTarget(TypeDesc thisType, TypeDesc otherType)
=> false;

private static bool IsCanonicalTypeArgMatch(TypeDesc type, TypeDesc otherType)
=> false;

private static bool IsCanonEquivalent(TypeDesc thisType, TypeDesc otherType)
=> false;
}
}
14 changes: 11 additions & 3 deletions src/coreclr/tools/Common/TypeSystem/Common/CastingHelper.cs
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,11 @@ private static bool CanCastToInternal(this TypeDesc thisType, TypeDesc otherType
return true;
}

if (IsCanonicalCastTarget(thisType, otherType))
{
return true;
}

switch (thisType.Category)
{
case TypeFlags.GenericParameter:
Expand Down Expand Up @@ -418,14 +423,14 @@ private static bool CanCastToInterface(this TypeDesc thisType, TypeDesc otherTyp

private static bool CanCastToNonVariantInterface(this TypeDesc thisType, TypeDesc otherType)
{
if (otherType.IsEquivalentTo(thisType))
if (otherType.IsEquivalentTo(thisType) || IsCanonEquivalent(thisType, otherType))
{
return true;
}

foreach (var interfaceType in thisType.RuntimeInterfaces)
{
if (interfaceType.IsEquivalentTo(otherType))
if (interfaceType.IsEquivalentTo(otherType) || IsCanonEquivalent(interfaceType, otherType))
{
return true;
}
Expand Down Expand Up @@ -469,6 +474,9 @@ private static bool CanCastByVarianceToInterfaceOrDelegate(this TypeDesc thisTyp

if (!arg.IsEquivalentTo(targetArg))
{
if (IsCanonicalTypeArgMatch(arg, targetArg))
continue;

GenericVariance variance = arrayCovariance
? GenericVariance.Covariant : ((GenericParameterDesc)instantiationOpen[i]).Variance;

Expand Down Expand Up @@ -541,7 +549,7 @@ private static bool CanCastToClass(this TypeDesc thisType, TypeDesc otherType, S

do
{
if (curType.IsEquivalentTo(otherType))
if (curType.IsEquivalentTo(otherType) || IsCanonEquivalent(curType, otherType))
return true;

curType = curType.BaseType;
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.

using System.Diagnostics;

Comment thread
MichalStrehovsky marked this conversation as resolved.
namespace Internal.TypeSystem
{
public static partial class TypeSystemConstraintsHelpers
{
private static bool IsSpecialTypeMeetingConstraint(TypeDesc type, GenericConstraints constraint)
=> false;

private static bool CanCastToConstraintWithCanon(TypeDesc instantiationParam, TypeDesc instantiatedConstraintType)
=> false;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ public InstantiationContext(Instantiation typeInstantiation, Instantiation metho
}
}

public static class TypeSystemConstraintsHelpers
public static partial class TypeSystemConstraintsHelpers
{
private static bool VerifyGenericParamConstraint(InstantiationContext genericParamContext, GenericParameterDesc genericParam,
InstantiationContext instantiationParamContext, TypeDesc instantiationParam)
Expand All @@ -29,23 +29,26 @@ private static bool VerifyGenericParamConstraint(InstantiationContext genericPar
if ((constraints & GenericConstraints.ReferenceTypeConstraint) != 0)
{
if (!instantiationParam.IsGCPointer
&& !CheckGenericSpecialConstraint(instantiationParam, GenericConstraints.ReferenceTypeConstraint))
&& !CheckGenericSpecialConstraint(instantiationParam, GenericConstraints.ReferenceTypeConstraint)
&& !IsSpecialTypeMeetingConstraint(instantiationParam, GenericConstraints.ReferenceTypeConstraint))
return false;
}

// Check default constructor constraint
if ((constraints & GenericConstraints.DefaultConstructorConstraint) != 0)
{
if (!instantiationParam.HasExplicitOrImplicitDefaultConstructor()
&& !CheckGenericSpecialConstraint(instantiationParam, GenericConstraints.DefaultConstructorConstraint))
&& !CheckGenericSpecialConstraint(instantiationParam, GenericConstraints.DefaultConstructorConstraint)
&& !IsSpecialTypeMeetingConstraint(instantiationParam, GenericConstraints.DefaultConstructorConstraint))
return false;
}

// Check struct constraint
if ((constraints & GenericConstraints.NotNullableValueTypeConstraint) != 0)
{
if ((!instantiationParam.IsValueType || instantiationParam.IsNullable)
&& !CheckGenericSpecialConstraint(instantiationParam, GenericConstraints.NotNullableValueTypeConstraint))
&& !CheckGenericSpecialConstraint(instantiationParam, GenericConstraints.NotNullableValueTypeConstraint)
&& !IsSpecialTypeMeetingConstraint(instantiationParam, GenericConstraints.NotNullableValueTypeConstraint))
return false;
}

Expand All @@ -62,6 +65,9 @@ private static bool VerifyGenericParamConstraint(InstantiationContext genericPar
if (CanCastConstraint(ref instantiatedConstraints, instantiatedType))
continue;

if (CanCastToConstraintWithCanon(instantiationParam, instantiatedType))
continue;

// CanCastTo below assumes thisType is boxed and that allows additional cases
// to be considered castable. But int is not castable to Nullable<int> and neither are enums.
if (instantiationParam.IsValueType && instantiatedType.IsValueType && !instantiationParam.IsEquivalentTo(instantiatedType))
Expand Down
6 changes: 6 additions & 0 deletions src/coreclr/tools/ILVerification/ILVerification.projitems
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@
<Compile Include="$(ToolsCommonPath)TypeSystem\Common\CastingHelper.cs">
<Link>TypeSystem\Common\CastingHelper.cs</Link>
</Compile>
<Compile Include="$(ToolsCommonPath)TypeSystem\Common\CastingHelper.NonCanon.cs">
<Link>TypeSystem\Common\CastingHelper.NonCanon.cs</Link>
</Compile>
<Compile Include="$(ToolsCommonPath)TypeSystem\Common\FunctionPointerType.cs">
<Link>TypeSystem\Common\FunctionPointerType.cs</Link>
</Compile>
Expand Down Expand Up @@ -372,5 +375,8 @@
<Compile Include="$(ToolsCommonPath)TypeSystem\Common\TypeSystemConstraintsHelpers.cs">
<Link>TypeSystem\Common\TypeSystemConstraintsHelpers.cs</Link>
</Compile>
<Compile Include="$(ToolsCommonPath)TypeSystem\Common\TypeSystemConstraintsHelpers.NonCanon.cs">
<Link>TypeSystem\Common\TypeSystemConstraintsHelpers.NonCanon.cs</Link>
</Compile>
</ItemGroup>
</Project>
Original file line number Diff line number Diff line change
Expand Up @@ -783,6 +783,10 @@ private sealed class MakeGenericTypeSite : INodeWithRuntimeDeterminedDependencie
{
var list = new DependencyList();
TypeDesc instantiatedType = _type.InstantiateSignature(typeInstantiation, methodInstantiation);

// InstantiateSignature could end up with a denormalized shape (Foo<object, __Canon>) so normalize.
instantiatedType = instantiatedType.NormalizeInstantiation();

if (instantiatedType.CheckConstraints(new InstantiationContext(typeInstantiation, methodInstantiation)))
RootingHelpers.TryGetDependenciesForReflectedType(ref list, factory, instantiatedType, "MakeGenericType");
return list;
Expand Down
Loading
Loading