Skip to content
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

[wasm] Set WasmEnableWebcil to true by default #86330

Merged
merged 2 commits into from
May 18, 2023

Conversation

lambdageek
Copy link
Member

@lambdageek lambdageek commented May 16, 2023

Switch to webcil by default in WebAssembly.

Contributes to #80807

Also fix some debugger test failures.

@lambdageek lambdageek mentioned this pull request May 16, 2023
5 tasks
@lambdageek
Copy link
Member Author

the symbol server debugger tests are failing with a pdb checksum mismatch. which is weird because we're apparently reading the debug directory data correctly. but somehow it is wrong

When using reflection to call a method on BlobReader, make sure to
copy the struct after the call since the internal state is modified

Fixes PdbChecksum extraction from Webcil files

(This was a problem even before webcil-in-wasm)
@lambdageek lambdageek marked this pull request as ready for review May 16, 2023 20:20
@lambdageek lambdageek added the arch-wasm WebAssembly architecture label May 16, 2023
@ghost
Copy link

ghost commented May 16, 2023

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Switch to webcil by default in WebAssembly.

Contributes to #80807

Also fix some debugger test failures.

Author: lambdageek
Assignees: lambdageek
Labels:

arch-wasm, area-Build-mono

Milestone: -

@lambdageek
Copy link
Member Author

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lambdageek
Copy link
Member Author

@maraf @radical this is ready for review. the multi-threaded debugger issues are known issues. single-threaded debugging is passing with webcil-in-wasm now

Copy link
Member

@maraf maraf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

@lambdageek
Copy link
Member Author

I'm going to hold off on hitting "merge" until I try some aspnetcore and SDK AOT tests locally to get an idea of how much downstream work there will be

@lambdageek
Copy link
Member Author

@maraf will work on downstream test fixes for dotnet/sdk and dotnet/aspnetcore. We're going to merge this PR.

@lambdageek lambdageek merged commit 7b0d8fe into dotnet:main May 18, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jun 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-Build-mono
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants