Open
Conversation
bangtianliu
force-pushed
the
fusilli-tuner
branch
8 times, most recently
from
e8d2f38 to
eb387a7
Compare
bangtianliu
requested review from
Max191,
RattataKing,
kuhar and
sjain-stanford
bangtianliu
force-pushed
the
fusilli-tuner
branch
from
eb387a7 to
5940a80
Compare
kuhar
reviewed
Feb 9, 2026
Member
kuhar
left a comment
kuhar
left a comment
There was a problem hiding this comment.
Overall looks OK.
@bangtianliu can you confirm what the code coverage currently is and if we have some function that could use tests.
bangtianliu
force-pushed
the
fusilli-tuner
branch
from
5940a80 to
2275e69
Compare
bangtianliu
force-pushed
the
fusilli-tuner
branch
2 times, most recently
from
9a0da6f to
56dca27
Compare
Contributor
Author
|
cc @kuhar for another look |
bangtianliu
force-pushed
the
fusilli-tuner
branch
from
56dca27 to
8b4d1fe
Compare
kuhar
reviewed
Feb 20, 2026
Member
Have you checked this? |
Signed-off-by: Bangtian Liu <liubangtian@gmail.com>
…d improve code clarity Signed-off-by: Bangtian Liu <liubangtian@gmail.com>
bangtianliu
force-pushed
the
fusilli-tuner
branch
from
5f34e6b to
74ff8a2
Compare
Contributor
Author
Yes. Just added more tests in the latest commit. |
Signed-off-by: Bangtian Liu <liubangtian@gmail.com>
bangtianliu
force-pushed
the
fusilli-tuner
branch
from
74ff8a2 to
f1f1615
Compare
RattataKing
reviewed
Feb 20, 2026
RattataKing
reviewed
Feb 20, 2026
Signed-off-by: Bangtian Liu <liubangtian@gmail.com>
Contributor
Author
|
cc @kuhar for another look. |
bangtianliu
force-pushed
the
fusilli-tuner
branch
3 times, most recently
from
cdd3f02 to
ca5058b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a new tuner for operations supported by Fusilli.
The Fusilli tuner follows a similar workflow to the existing BOO Tuner:
fusilli_benchmark_driver --dumpto generate source MLIR and compile commands--iree-scheduling-dump-statistics-*) from compile commandsroot_opattributes through adding--iree-config-add-tuner-attributesand--iree-hal-dump-executable-benchmarks-toto the compile commands.Also attached the fusilli tuning log: https://gist.github.com/bangtianliu/bb79ad54f23bf6cec9005fefb99638f7
Assisted-by: Claude Code