Skip to content

Disable IREE benchmarks in fusilli's captive IREE build#422

Merged
sjain-stanford merged 1 commit into
iree-org:mainfrom
AaronStGeorge:p048-asan-build
May 15, 2026
Merged

Disable IREE benchmarks in fusilli's captive IREE build#422
sjain-stanford merged 1 commit into
iree-org:mainfrom
AaronStGeorge:p048-asan-build

Conversation

@AaronStGeorge
Copy link
Copy Markdown
Contributor

@AaronStGeorge AaronStGeorge commented May 14, 2026

Motivation

In ASan builds of TheRock, an issue in IREE's third_party/benchmark submodule breaks fusilli's CMake configure during IREE's sub-configure. A compiler invocation, inside a capability check, inside IREE's captive benchmark dependency, fails to find the ASan runtime library. This issue would likely only happen in TheRock when using the compiler hermetically created as part of the build.

Given that fusilli-provider and fusilli don't use any code paths that require IREE's captive benchmark dependency the easiest path to fix the issue is simply to sidestep it by removing the dependency.

Technical Details

PR iree-org/iree#24167 added an IREE_BUILD_BENCHMARKS option. When IREE_BUILD_BENCHMARKS and IREE_BUILD_TESTS are both set to OFF, IREE doesn't require third_party/benchmark. A follow up PR will remove the dep from fusilli's docker here.

IREE PR #24167 (Enable building without google benchmark dependency) added the IREE_BUILD_BENCHMARKS option. Set it OFF here so fusilli's captive IREE doesn't pull google-benchmark via third_party/benchmark.
AaronStGeorge added a commit to AaronStGeorge/docker that referenced this pull request May 14, 2026
After iree-org/fusilli#422 fusilli will no longer require IREE to have `third_party/benchmark`.

We may want merge this and bump the docker container as part of iree-org/fusilli#422.
@sjain-stanford sjain-stanford merged commit b68364d into iree-org:main May 15, 2026
22 of 26 checks passed
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