-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MT][browser] WithExecutionMode
fails
#91661
Comments
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsFailing after enabling all MT library tests (#91536).
|
Does this still happen after #94214 ? |
The test fails with
At this assert:
The issue here is that the This is result of The draft #96215 fixes this. The PLinq doesn't build yet through, as there are missing pieces. Let try to complete these in the draft. |
Add System.Threading.WebAssembly.Threading ref assembly to enable more parts of threading API on wasm Fix System.Linq.Parallel assembly to work as MT on wasm Increase the initial thread pool size to 7 to let the System.Linq.Parallel tests pass Fixes #91661 * [wasm] Enable more MT PLinq areas Also fix the csproj condition to detect FeatureWasmThreads * Add missing pieces * Add System.Threading.WebAssembly.Threading.csproj * Fix System.Threading to work with the new wasm MT ref assembly * Enable the failing test * Use the new ref assembly in the System.Linq.Parallel * Remove now obsolete API check supressions * Increase the thread pool size to make the plinq tests pass * Add API check suppressions for System.Threading * Always use the API suppressions for System.Threading * Remove unwanted attributes from the implementation * Add suppressions to corelib * Revert "Remove unwanted attributes from the implementation" This reverts commit aedafd3. * Build fixes * Remove unnecessary suppression * Remove unwanted source file from the list
Add System.Threading.WebAssembly.Threading ref assembly to enable more parts of threading API on wasm Fix System.Linq.Parallel assembly to work as MT on wasm Increase the initial thread pool size to 7 to let the System.Linq.Parallel tests pass Fixes dotnet#91661 * [wasm] Enable more MT PLinq areas Also fix the csproj condition to detect FeatureWasmThreads * Add missing pieces * Add System.Threading.WebAssembly.Threading.csproj * Fix System.Threading to work with the new wasm MT ref assembly * Enable the failing test * Use the new ref assembly in the System.Linq.Parallel * Remove now obsolete API check supressions * Increase the thread pool size to make the plinq tests pass * Add API check suppressions for System.Threading * Always use the API suppressions for System.Threading * Remove unwanted attributes from the implementation * Add suppressions to corelib * Revert "Remove unwanted attributes from the implementation" This reverts commit aedafd3. * Build fixes * Remove unnecessary suppression * Remove unwanted source file from the list
Failing after enabling all MT library tests (#91536).
The problematic operation is
SelectMany
.The text was updated successfully, but these errors were encountered: