Fix LibraryImport QCall signature mismatches

[davidwrighton]

Summary

• Correct assembly tracing LibraryImport declarations that returned bool even though their QCall implementations return void.
• Align [MarshalAs(UnmanagedType.Bool)] parameters with 4-byte native BOOL parameters for thread-static helpers, assembly load tracing, and ThreadPool I/O enqueue events.
• Keep the enabled and disabled NativeAOT EventPipe implementations consistent; Mono's separate InternalCall path already uses the correct MonoBoolean ABI.

These mismatches could read an undefined return value or pass a differently sized boolean across the managed/native ABI, depending on the target architecture and calling convention.

Testing

• build.cmd clr -rc Debug
• build.cmd clr.aot+libs -rc Debug -lc Release
• ThreadStatic01 and ThreadStatic06 on the rebuilt CoreCLR
• System.Threading.ThreadPool.Tests: 70 passed, 1 skipped; the Windows ThreadPool-specific ReadWriteAsyncTest was skipped because its runtime condition was not met

Note

This pull request description was generated with GitHub Copilot.

[azure-pipelines]

Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

[dotnet-policy-service]

Tagging subscribers to this area: @agocke
See info in area-owners.md if you want to be subscribed.

[Copilot]

Pull request overview

This PR aligns several CoreCLR/NativeAOT QCall boundaries so the managed LibraryImport signatures match the native implementations, avoiding ABI mismatches (void-vs-bool returns and 1-byte bool vs 4-byte BOOL parameters) in tracing and thread-static helpers.

Changes:

• Updated native QCall signatures to use 4-byte BOOL for boolean parameters where managed marshalling uses UnmanagedType.Bool.
• Fixed CoreCLR LibraryImport assembly tracing declarations that previously returned bool even though the native QCalls return void.
• Kept NativeAOT enabled/disabled EventPipe implementations consistent for the ThreadPool I/O enqueue event.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/coreclr/vm/nativeeventsource.h	Changes NativeRuntimeEventSource_LogThreadPoolIOEnqueue boolean parameter type to BOOL to match managed marshalling.
src/coreclr/vm/nativeeventsource.cpp	Updates the corresponding QCall implementation signature to BOOL for multiDequeues.
src/coreclr/vm/JitQCallHelpers.h	Changes thread-static helper QCall declarations to take BOOL instead of bool for ABI consistency.
src/coreclr/vm/jithelpers.cpp	Updates thread-static helper QCall implementations to accept BOOL for gcStatic.
src/coreclr/vm/assemblynative.hpp	Changes AssemblyNative_TraceAssemblyLoadFromResolveHandlerInvoked to take BOOL for isTrackedAssembly.
src/coreclr/vm/assemblynative.cpp	Updates the corresponding QCall implementation signature to use BOOL.
src/coreclr/System.Private.CoreLib/src/System/Runtime/Loader/AssemblyLoadContext.CoreCLR.cs	Fixes tracing LibraryImport declarations to return void (matching native QCalls) and removes incorrect return marshalling.
src/coreclr/nativeaot/Runtime/disabledruntimeeventinternal.cpp	Updates disabled NativeAOT event stub for LogThreadPoolIOEnqueue to use BOOL for the boolean parameter.