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.

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

Add compiled/cached MachineCode #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

OneOfOne
Copy link

@OneOfOne OneOfOne commented May 11, 2021

$ /bin/go test -run=xxxx -bench=Parse -benchmem -benchtime 5s

goos: linux
goarch: amd64
pkg: github.com/bspaans/jit-compiler/ir
cpu: AMD Ryzen Threadripper 2950X 16-Core Processor 
Benchmark_Parse/No-SSA/No-Cache-32         	      19	 307234346 ns/op	188360269 B/op	 6094822 allocs/op
Benchmark_Parse/SSA/Cache-32               	  411292	     15084 ns/op	       0 B/op	       0 allocs/op
Benchmark_Parse/SSA/No-Cache-32            	      18	 335270616 ns/op	188984950 B/op	 6107401 allocs/op
Benchmark_Parse/No-SSA/Cache-32            	  401347	     15060 ns/op	       0 B/op	       0 allocs/op
PASS
ok  	github.com/bspaans/jit-compiler/ir	27.835s

- Added some benchmarks

```sh
goos: linux
goarch: amd64
pkg: github.com/bspaans/jit-compiler/ir
cpu: AMD Ryzen Threadripper 2950X 16-Core Processor
Benchmark_Parse/No-SSA/No-Cache-32         	       4	 315428400 ns/op	188349976 B/op	 6094850 allocs/op
Benchmark_Parse/SSA/Cache-32               	   80162	     15728 ns/op	       0 B/op	       0 allocs/op
Benchmark_Parse/SSA/No-Cache-32            	       4	 321301980 ns/op	188934464 B/op	 6107397 allocs/op
Benchmark_Parse/No-SSA/Cache-32            	   76292	     15752 ns/op	       0 B/op	       0 allocs/op
PASS
ok  	github.com/bspaans/jit-compiler/ir	10.452s
```
@MatejMagat305
Copy link
Contributor

why not be this improvements accept?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants