Skip to content

[Wasm RyuJit] bitcast#124138

Open
AndyAyersMS wants to merge 3 commits intodotnet:mainfrom
AndyAyersMS:WasmBitcast
Open

[Wasm RyuJit] bitcast#124138
AndyAyersMS wants to merge 3 commits intodotnet:mainfrom
AndyAyersMS:WasmBitcast

Conversation

@AndyAyersMS
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 8, 2026 02:38
@AndyAyersMS
Copy link
Member Author

@dotnet/jit-contrib PTAL

@github-actions github-actions bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Feb 8, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds initial WASM RyuJit codegen support for GT_BITCAST, enabling bit reinterpretation operations to be emitted as native Wasm reinterpret opcodes.

Changes:

  • Dispatch GT_BITCAST in CodeGen::genCodeForTreeNode.
  • Add CodeGen::genCodeForBitCast implementation in the WASM code generator to emit Wasm reinterpret instructions.

@am11 am11 added the arch-wasm WebAssembly architecture label Feb 8, 2026
@dotnet-policy-service
Copy link
Contributor

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

var_types fromType = tree->gtGetOp1()->TypeGet();

assert(toType == genActualType(tree));
assert(fromType == genActualType(tree->gtGetOp1()));
Copy link
Contributor

Choose a reason for hiding this comment

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

This assert won't hold. The operand can be of arbitrary type (incl. small ones). It needs to be genActualTyped explicitly.

Copilot AI review requested due to automatic review settings February 9, 2026 15:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch-wasm WebAssembly architecture area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants