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.

Sign up for GitHub

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

Jump to bottom

Optimize zlib for POWER8 on AT 12.0 #780

Closed
tuliom opened this issue Jan 16, 2019 · 5 comments · Fixed by #1547
Closed

Optimize zlib for POWER8 on AT 12.0 #780

tuliom opened this issue Jan 16, 2019 · 5 comments · Fixed by #1547
Assignees
@mscastanho
Labels
enhancement

Comments

@tuliom
Copy link
Contributor

tuliom commented Jan 16, 2019

Please include madler/zlib#335 in the zlib build in order to get the performance optimization from that PR.
@mscastanho
Copy link
Contributor

Is this still relevant now that #1341 has been merged? That PR applies the optimizations only to AT next (14.0), but we likely don't need it anymore for 12.0 anyway.

@tuliom
Copy link
Contributor Author

tuliom commented Apr 30, 2020 •
edited
Loading

I expected to have these optimizations at least on the last AT release (13.0).
By the time this issue was opened, it was AT 12.0.

@tuliom
Copy link
Contributor Author

tuliom commented Jul 10, 2020

@mscastanho I've just realized that #1341 has not been applied even to AT 14.0.

@mscastanho mscastanho self-assigned this Jul 10, 2020
@mscastanho
Copy link
Contributor

@tuliom true. I'll take care of that.

@mscastanho mscastanho mentioned this issue Jul 15, 2020
Merged
@Neustradamus
Copy link

Note: Now there is zlib 1.2.13 with CVE fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mscastanho mscastanho

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Apply zlib optimizations to AT 12, 13 and 14 mscastanho/advance-toolchain
3 participants
@tuliom @Neustradamus @mscastanho