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

PR CI (in-tree mlir-xten) #67

Merged
merged 7 commits into from
Aug 20, 2024
Merged

PR CI (in-tree mlir-xten) #67

merged 7 commits into from
Aug 20, 2024

Conversation

josel-amd
Copy link
Collaborator

@josel-amd josel-amd commented Aug 19, 2024

Build mlir-xten standalone. This is important to make the CI work since mlir-xten relies on llvm-project. Having llvm-project as a third-party dependency allows the repo to exist by itself. As a side-effect, this also adds documentation on how to build this project in the shape of a workflow file.

@josel-amd josel-amd force-pushed the jose.FXML-4901 branch 5 times, most recently from b6aa724 to 4b90648 Compare August 19, 2024 11:44
@josel-amd josel-amd changed the title Experiments with the CI PR CI (in-tree mlir-xten) Aug 19, 2024
@josel-amd josel-amd marked this pull request as ready for review August 19, 2024 11:52
@josel-amd
Copy link
Collaborator Author

I'll add dependabot in a different PR to bump the submodule.

@josel-amd
Copy link
Collaborator Author

josel-amd commented Aug 19, 2024

  • Move changes to .github/workflows/buildAndTest.yml

.github/workflows/build-llvm-image.yml Outdated Show resolved Hide resolved
.github/workflows/build-llvm-image.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@TinaAMD TinaAMD left a comment

Choose a reason for hiding this comment

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

LGTM

.github/workflows/build-llvm-image.yml Outdated Show resolved Hide resolved
@josel-amd
Copy link
Collaborator Author

Heads-up: This PR is now removing all other workflows, just keeping the buildAndTest.yml. Going to merge soon.

@josel-amd josel-amd merged commit 89facc7 into feature/fused-ops Aug 20, 2024
@josel-amd josel-amd deleted the jose.FXML-4901 branch August 20, 2024 11: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.

3 participants