-
Notifications
You must be signed in to change notification settings - Fork 350
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
feat: support aarch64 #645
Conversation
b4bbb87
to
d502e85
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #645 +/- ##
===========================================
- Coverage 77.56% 52.45% -25.11%
===========================================
Files 83 175 +92
Lines 11542 11374 -168
===========================================
- Hits 8952 5966 -2986
- Misses 2192 5085 +2893
+ Partials 398 323 -75 ☔ View full report in Codecov by Sentry. |
558fe5b
to
d980618
Compare
Benchmark Data for Decoder:
|
3abf63b
to
84e3754
Compare
|
b94c0a0
to
bd7e026
Compare
Compare with
|
Not batch allocate the map will cause 10% overhead here.
|
e2be1dd
to
75702b6
Compare
c71218e
to
bd14148
Compare
Rewrite the decoder and encoder to support aarch64. Not fallback into
encoding/json
in aarch64.Performance
compare with jit decoder/encoder:
Unmarshal/Interface: about 1.7x
Unmarshal/Concrete: 1.0x
Marshal/Interface: 0.7x
Marshal/Concrete: 0.6x
For more details pls see the data in follow comments.
Compatibility
there are no break changes here.