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

[µTVM] Remove need for -mcpu=native #7276

Merged
merged 1 commit into from
Jan 14, 2021

Conversation

areusch
Copy link
Contributor

@areusch areusch commented Jan 13, 2021

Fixes x86 builds on OS X (I.e. as done by tests/python/unittest/test_crt.py). We may need to revisit this compiler auto-select logic at some point as well, since -mcpu may not always be the best way to specify a target ISA. However, I'd like to ideally defer that past Project-level API from the M2 roadmap.

@areusch
Copy link
Contributor Author

areusch commented Jan 14, 2021

@tom-gall @tmoreau89 @tqchen

@areusch
Copy link
Contributor Author

areusch commented Jan 14, 2021

@gromero

Copy link
Contributor

@tmoreau89 tmoreau89 left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@tmoreau89 tmoreau89 merged commit f503d82 into apache:main Jan 14, 2021
masahi pushed a commit to masahi/tvm that referenced this pull request Jan 18, 2021
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Jan 20, 2021
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Jan 21, 2021
electriclilies pushed a commit to electriclilies/tvm that referenced this pull request Feb 18, 2021
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