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

Consume IREE as submodule by default. #496

Merged
merged 2 commits into from
May 19, 2022

Conversation

ingomueller-net
Copy link
Collaborator

@ingomueller-net ingomueller-net commented May 16, 2022

This PR changes the default build process such that IREE (and thus LLVM) is consumed as a git submodule instead of using a specific branch. This freezes the version of these dependencies such that unrelated updates of that branch do not affect them.

@ingomueller-net ingomueller-net force-pushed the iree-submodule branch 3 times, most recently from 6e733c6 to 33136a5 Compare May 16, 2022 14:02
@ingomueller-net ingomueller-net changed the title WIP: Consume IREE as submodule by default. Consume IREE as submodule by default. May 16, 2022
Copy link
Contributor

@nicolasvasilache nicolasvasilache left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

.gitmodules Outdated
@@ -0,0 +1,3 @@
[submodule "third_party/iree"]
path = third_party/iree
url = https://github.com/nicolasvasilache/iree.git
Copy link
Contributor

Choose a reason for hiding this comment

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

you could take the real iree directly, I was just using my fork because I was maintaining the proper hash there.

@ingomueller-net ingomueller-net merged commit ecf41d2 into iree-org:main May 19, 2022
@ingomueller-net ingomueller-net deleted the iree-submodule branch May 19, 2022 19:36
@ingomueller-net
Copy link
Collaborator Author

@webmiche: Note that the build process has changed with the landing of this PR. See the instructions in the README.

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