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.

Sign up for GitHub

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

Jump to bottom

Allow disabling -march=native without disabling intrinsics? #539

Closed
fingolfin opened this issue Apr 2, 2022 · 2 comments
Closed

Allow disabling -march=native without disabling intrinsics? #539

fingolfin opened this issue Apr 2, 2022 · 2 comments
Labels
minor A label for PRs or issues that are minor in some sense. resolved-pending-release A label for issues that have been resolved and that can be closed when a new version is released. technical A necessary technical change, not interesting mathematically/feature-wise

Comments

@fingolfin
Copy link
Contributor

For creating portable binaries, it is important to not use -march=native. But right now to disable this without patching, one has to use --with_intrinsics but that actually disables more things that are perfectly fine to use in portable binaries, yet slow down things.

So perhaps an option could be added which disabled only -march=native?

(To be honest, I would do the opposite and not use -march=native by default, but that's just me, clearly you have decided otherwise :-) ).
@james-d-mitchell
Copy link
Member

Thanks @fingolfin, I'll be happy to remove the -march=native by default, I'm not sure that this was a very well reasoned decision, and given the difficulties it's causing, better to just remove it.

@james-d-mitchell james-d-mitchell mentioned this issue Apr 6, 2022
Merged
@james-d-mitchell james-d-mitchell added minor A label for PRs or issues that are minor in some sense. technical A necessary technical change, not interesting mathematically/feature-wise resolved-pending-release A label for issues that have been resolved and that can be closed when a new version is released. labels Apr 6, 2022
@digraphs digraphs deleted a comment from github-actions bot Aug 5, 2022
@james-d-mitchell
Copy link
Member

This was removed in v1.5.3 I think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
minor A label for PRs or issues that are minor in some sense. resolved-pending-release A label for issues that have been resolved and that can be closed when a new version is released. technical A necessary technical change, not interesting mathematically/feature-wise
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fingolfin @james-d-mitchell