Skip to content

🐛 add memmove shim for wasm builds#32

Open
rauhryan wants to merge 1 commit into
mainfrom
rr/memmove-shim
Open

🐛 add memmove shim for wasm builds#32
rauhryan wants to merge 1 commit into
mainfrom
rr/memmove-shim

Conversation

@rauhryan
Copy link
Copy Markdown
Collaborator

Summary

  • add a freestanding memmove shim for wasm32 builds
  • declare memmove alongside the existing memcpy/memset shims
  • fixes CI linker failures where clang emits calls to memmove under -nostdlib

Validation

  • make CC=/opt/homebrew/opt/llvm/bin/clang
  • deno test

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 10, 2026

Open in StackBlitz

npm i https://pkg.pr.new/clayterm@32

commit: d386e17

rauhryan added a commit that referenced this pull request May 10, 2026
@cowboyd
Copy link
Copy Markdown
Contributor

cowboyd commented May 11, 2026

I'm going to approve this, but curious what code is using it?

We need to get a performance suite in here so that we don't end up accidentally degrading performance.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants