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

ARM support #38

Closed
wants to merge 1 commit into from
Closed

ARM support #38

wants to merge 1 commit into from

Conversation

fperrad
Copy link

@fperrad fperrad commented Nov 6, 2015

this patch removes SSE which are incompatible with ARM architecture

g++: error: unrecognized command line option -mfpmath=sse
g++: error: unrecognized command line option -msse3

see full log: http://www.cpantesters.org/cpan/report/58cbce1a-83a6-11e5-a4a8-dfc10b3facc5

remove SSE which are incompatible with ARM architecture
@wbraswell
Copy link
Owner

@fperrad
Hello, and thank you very much for your interest in RPerl!
Your patch completely disables SSE, which is not good because we rely on SSE for some RPerl optimizations.
I suspect the long-term solution will be a fully-automated SSE controller which can enable or disable SSE based on multiple factors, including hardware support or user preference.
For now, I think the first step is to add simple if() conditional statements to your patch which will only disable SSE if it finds an ARM architecture.
Can you please update your code and make another pull request?

@fperrad
Copy link
Author

fperrad commented Nov 8, 2015

superseded by #39

@fperrad fperrad closed this Nov 8, 2015
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