-
Notifications
You must be signed in to change notification settings - Fork 202
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
Update README.md #429
Update README.md #429
Conversation
Advanced SIMD (aka NEON) is mandatory for AArch64. There is no need for adding extra options to the compiler for 64 bit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
Do you mind to add your 64bit comment in the README as well? This might be helpful for new users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The README file needs some work ; I see a typo (likely more than 1) and some errors with respect to macOS. We need a section for how to handle Apple ARM64 / Aarch64. This change is fine, but I agree with @jdemel that I'd like to see your comment herein noted in the README file itself. @JamMarHer can you make this change? If not I'll look into a new PR that does this & more. Thx!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Thanks for your contribution!
@michaelld do we need these commits to be signed off? |
I think these are good to go as-is. They are small and not to any code. Not sure if that's what you were going for. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; thx!
I'm going to go ahead and merge based on my prior comment. |
Update README.md
Advanced SIMD (aka NEON) is mandatory for AArch64.
There is no need for adding extra options to the compiler for 64 bit.