Fix RuntimeMethodHandle cache key for async variants

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.

[Copilot]

Fixes#128286
Fixes#128291

NativeAOT TypeLoader was interning dynamically created RuntimeMethodHandles without including the async-variant bit in the key. This could cause async/non-async lookups for the same method components to alias to one cached handle and resolve the wrong method variant; this change makes async variant part of handle identity and re-enables the affected System.Text.Json source-gen test project.

• TypeLoader runtime method handle key

  • Extended RuntimeMethodHandleKey to include isAsyncVariant in identity.
  • Updated constructor and key creation to pass/store isAsyncVariant.
  • Updated Equals to require matching async-variant state.
  • Updated GetHashCode to include async-variant state via VersionResilientHashCode.CombineTwoValuesIntoHash.
  • Added the required explicit uncheckedint/uint casts around the hash combiner call so the NativeAOT TypeLoader build compiles correctly.

• NativeAOT libraries test project inclusion

  • Removed the NativeAOT exclusion entry for:
    • System.Text.Json/tests/System.Text.Json.SourceGeneration.Tests/System.Text.Json.SourceGeneration.Roslyn4.4.Tests.csproj
  • Kept unrelated platform-specific exclusions intact.

// before: async flag omitted from interning keyRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs);// after: async flag is part of key identityRuntimeMethodHandleKeykey=newRuntimeMethodHandleKey(declaringTypeHandle,handle,genericMethodArgs,isAsyncVariant);

This pull request was created from Copilot chat.
>

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull request overview

This PR updates NativeAOT’s TypeLoader RuntimeMethodHandle interning to treat the “async variant” bit as part of a handle’s identity, preventing cache aliasing between async/non-async variants. It also re-enables the previously excluded System.Text.Json.SourceGeneration.Roslyn4.4.Tests project for most NativeAOT configurations by removing the broad exclusion while keeping existing OS/arch-specific exclusions.

Changes:

• Extended RuntimeMethodHandleKey to include the async-variant flag in equality and hashing, and passed the flag when creating interning keys.
• Updated hashing logic for RuntimeMethodHandleKey to incorporate the async-variant flag.
• Removed the unconditional NativeAOT test exclusion for System.Text.Json.SourceGeneration.Roslyn4.4.Tests (while retaining existing conditional exclusions, e.g., OS/arch-specific cases).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/nativeaot/System.Private.TypeLoader/src/Internal/Runtime/TypeLoader/TypeLoaderEnvironment.LdTokenResultLookup.cs	Adds isAsyncVariant into the runtime method handle interning key (equals/hash + key construction).
src/libraries/tests.proj	Re-enables System.Text.Json.SourceGeneration.Roslyn4.4.Tests for NativeAOT by removing the broad exclusion entry.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[MichalStrehovsky]

/azp run runtime-nativeaot-outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[eduardo-vp]

LGTM! Just looks like Copilot removed from the PR description the issues that this PR fixes.