-
Notifications
You must be signed in to change notification settings - Fork 349
add __ARM_NEON support #157
base: master
Are you sure you want to change the base?
Conversation
|
Performance gain is not huge currently, but most of the instructions are just literally translated, yet, and not iterated upon.
EDIT: typos; resolved |
|
EDIT: resolved |
|
Benchmark on C6g.4xlarge with clang-10 and ARCH_FLAGS=-mcpu=neoverse-n1 |
|
Please use an ‘isb’ not a ‘dmb’. See this as an example. haproxy/haproxy@1e237d0 |
AGSaidi
left a comment
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 making the isb change, lgtm.
|
Looks good to me. |
lepton-scalar