Skip to content

cheddar: add optional scale-snu CHEDDAR dependency - #3334

Open
AlexanderViand wants to merge 1 commit into
google:mainfrom
AlexanderViand:alex/cheddar-dependency
Open

cheddar: add optional scale-snu CHEDDAR dependency#3334
AlexanderViand wants to merge 1 commit into
google:mainfrom
AlexanderViand:alex/cheddar-dependency

Conversation

@AlexanderViand

@AlexanderViand AlexanderViand commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

PR 3/? on Cheddar :)

This just adds an optional cheddar bazel dependency (EDIT: only expected to work on linux with CUDA)

@AlexanderViand
AlexanderViand requested a review from j2kun August 13, 2026 17:58
Comment thread tools/heir-opt.bzl Outdated
Comment thread MODULE.bazel Outdated
@AlexanderViand
AlexanderViand force-pushed the alex/cheddar-dependency branch 3 times, most recently from a541705 to 8619f38 Compare August 17, 2026 08:37
@AlexanderViand
AlexanderViand requested a review from j2kun August 17, 2026 08:48
@AlexanderViand
AlexanderViand force-pushed the alex/cheddar-dependency branch 3 times, most recently from ab64f51 to 2d7eda3 Compare August 17, 2026 15:10

@j2kun j2kun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, my plan is to set up buildkite in September so that we can run the cheddar tests on GPU.

Comment thread patches/mlir_forward_call_result_out_params.patch Outdated
Comment thread patches/cheddar.patch

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Optional: can we also try to upstream the C++ changes here?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I'll open a PR against scale-snu/cheddar

@AlexanderViand
AlexanderViand force-pushed the alex/cheddar-dependency branch from 2d7eda3 to 3e6946c Compare August 17, 2026 16:32
@j2kun j2kun added the pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing label Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants