-
Notifications
You must be signed in to change notification settings - Fork 352
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 |
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