Skip to content

chore: migrate rust/simd to icp-cli#1404

Draft
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-rust-simd-to-icp-cli
Draft

chore: migrate rust/simd to icp-cli#1404
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-rust-simd-to-icp-cli

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Summary

  • Replaces dfx.json with icp.yaml using @dfinity/rust@v3.3.0
  • Moves source from mat_mat_mul/ into backend/ subdirectory; renames package to backend
  • Updates ic-cdk to 0.20 and replaces the separate ic-cdk-macros crate with the macros re-exported from ic_cdk
  • Preserves .cargo/config.toml with the wasm32-unknown-unknown SIMD target-feature comments (critical for the per-function #[target_feature(enable = "simd128")] approach this example demonstrates)
  • Rewrites Makefile with numbered icp canister call --query tests for all 7 public methods
  • Updates README.md with icp-cli deploy instructions, preserving the SIMD performance insights (10x f32 speedup, 2x u32 speedup) and explanation of the packing technique
  • Adds .github/workflows/simd.yml CI job using ghcr.io/dfinity/icp-dev-env-rust:1.0.0
  • Deletes dfx.json, BUILD.md, .devcontainer/

Test plan

  • icp network start -d && icp deploy builds and deploys the backend canister
  • make test runs all 7 SIMD benchmark query calls and verifies non-zero instruction counts
  • CI job rust-simd passes on the PR

🤖 Generated with Claude Code

- Replace dfx.json with icp.yaml using @dfinity/rust@v3.3.0
- Move mat_mat_mul/ source into backend/ subdirectory
- Rename package to backend, update ic-cdk to 0.20, replace ic_cdk_macros with ic_cdk
- Update Makefile with icp canister call --query tests for all 7 methods
- Update README with icp-cli deploy instructions, preserve SIMD performance insights
- Add .github/workflows/simd.yml CI job (icp-dev-env-rust:1.0.0)
- Delete dfx.json, BUILD.md, .devcontainer/, mat_mat_mul/
- Keep .cargo/config.toml with wasm32 SIMD target-feature comments

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant