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

Correcting usage of --iree-hal-target-device=llvm-cpu. #1122

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

benvanik
Copy link
Collaborator

@benvanik benvanik commented Mar 19, 2025

iree-org/iree#20295 will remove the legacy compatibility hack that made this flag work. Due to a compiler API issue it must land first and this PR should land once it has been integrated.

Fixes #1119.

@benvanik benvanik force-pushed the users/benvanik/fix-target-device branch from 0fadc7e to ef13f4a Compare March 21, 2025 00:30
@benvanik benvanik enabled auto-merge (squash) March 21, 2025 00:30
@benvanik
Copy link
Collaborator Author

iree-org/iree#20295 has landed in IREE. I'm leaving this here for someone on the project to land whenever it makes sense.

Copy link
Member

@ScottTodd ScottTodd left a comment

Choose a reason for hiding this comment

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

The changes LGTM. Looks like we may need to merge together with a version update like #1126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace usage of --iree-hal-target-device=llvm-cpu with local.
3 participants