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

Optionally compile wasmtime-bench-api with wasi-nn and wasi-crypto #2677

Merged
merged 1 commit into from
Feb 23, 2021

Conversation

abrown
Copy link
Contributor

@abrown abrown commented Feb 23, 2021

This adds the ability to add feature flags (e.g. --features wasi-nn) when compiling wasmtime-bench-api to allow benchmarking Wasmtime with WASI proposals included. Note that due to rust-lang/cargo#5364, these features are only available:

  • in the crates/bench-api directory, e.g. pushd crates/bench-api; cargo build --features wasi-crypto
  • or from the top-level project directory using -Zpackage-features, e.g. OPENVINO_INSTALL_DIR=/opt/intel/openvino cargo +nightly build -p wasmtime-bench-api -Zpackage-features --features wasi-nn

This adds the ability to add feature flags (e.g. `--features wasi-nn`) when compiling `wasmtime-bench-api` to allow benchmarking Wasmtime with WASI proposals included. Note that due to rust-lang/cargo#5364, these features are only available:
 - in the `crates/bench-api` directory, e.g. `pushd crates/bench-api; cargo build --features wasi-crypto`
 - or from the top-level project directory using `-Zpackage-features`, e.g. `OPENVINO_INSTALL_DIR=/opt/intel/openvino cargo +nightly build -p wasmtime-bench-api -Zpackage-features --features wasi-nn`
@abrown abrown requested a review from fitzgen February 23, 2021 01:14
@alexcrichton alexcrichton merged commit 96556ed into bytecodealliance:main Feb 23, 2021
@abrown abrown deleted the bench-wasi-proposals branch February 23, 2021 15:26
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