abrown opened PR #4096 from parse-bench-api-flags
to main
:
As discussed in a previous Wasmtime meeting, we need a way to be able to configure Wasmtime when running it in the Sightglass benchmark infrastructure. The easiest way to do this seemed to pass a string from Sightglass to the bench-api library and parse this in the same way that Wasmtime parses its CLI flags. The structure that contains these flags is
CommonOptions
, so it has been moved to its own crate to be depended on by bothwasmtime-cli
andwasmtime-bench-api
. Also, this change adds an externally-visible function for parsing a string intoCommonOptions
, which is used for configuring an engine.
abrown requested alexcrichton for a review on PR #4096.
abrown updated PR #4096 from parse-bench-api-flags
to main
.
abrown updated PR #4096 from parse-bench-api-flags
to main
.
abrown updated PR #4096 from parse-bench-api-flags
to main
.
abrown has marked PR #4096 as ready for review.
abrown updated PR #4096 from parse-bench-api-flags
to main
.
alexcrichton submitted PR review.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
Could this be changed to
listenfd
since that's the root dep we want to not warn about?
abrown updated PR #4096 from parse-bench-api-flags
to main
.
abrown updated PR #4096 from parse-bench-api-flags
to main
.
abrown updated PR #4096 from parse-bench-api-flags
to main
.
abrown merged PR #4096.
Last updated: Dec 23 2024 at 12:05 UTC