Skip to content

perf: Add AES key caching#189

Merged
bodgit merged 3 commits intomainfrom
aes7z-performance
Apr 1, 2024
Merged

perf: Add AES key caching#189
bodgit merged 3 commits intomainfrom
aes7z-performance

Conversation

@bodgit
Copy link
Owner

@bodgit bodgit commented Apr 1, 2024

Before:

$ go test -v -run '^$' -bench='AES'
goos: darwin
goarch: amd64
pkg: github.com/bodgit/sevenzip
cpu: Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
BenchmarkAES7z
BenchmarkAES7z-12    	       2	 533112205 ns/op
PASS
ok  	github.com/bodgit/sevenzip	1.808s

After:

$ go test -v -run '^$' -bench='AES'
goos: darwin
goarch: amd64
pkg: github.com/bodgit/sevenzip
cpu: Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
BenchmarkAES7z
BenchmarkAES7z-12    	    1167	   1063848 ns/op
PASS
ok  	github.com/bodgit/sevenzip	2.427s

Fixes #187

@bodgit bodgit added the enhancement New feature or request label Apr 1, 2024
@bodgit bodgit self-assigned this Apr 1, 2024
@coveralls
Copy link

coveralls commented Apr 1, 2024

Coverage Status

coverage: 75.106% (+0.08%) from 75.024%
when pulling df6194b on aes7z-performance
into 31f77ef on main.

@bodgit bodgit force-pushed the aes7z-performance branch from c6947f5 to df6194b Compare April 1, 2024 22:52
@bodgit bodgit merged commit 3d794c2 into main Apr 1, 2024
@bodgit bodgit deleted the aes7z-performance branch April 1, 2024 23:11
@bodgit bodgit mentioned this pull request Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

low performance for aes7z

2 participants