Skip to content

Conversation

herndlm
Copy link
Contributor

@herndlm herndlm commented Feb 15, 2023

was added in 28c5ff6, but everything seems to be still fine without it. not sure yet if this changes something performance-wise, I hope CI tells me

@herndlm herndlm changed the base branch from 1.10.x to 1.9.x February 15, 2023 09:09
@herndlm
Copy link
Contributor Author

herndlm commented Feb 15, 2023

tried to benchmark it, but my system is not good.. I think there's basically no performance difference

before

hyperfine 'make phpstan'
Benchmark 1: make phpstan
  Time (mean ± σ):     50.324 s ±  4.197 s    [User: 328.346 s, System: 22.814 s]
  Range (min … max):   47.511 s … 59.977 s    10 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

I re-ran this 3 times, but always had some outliers and gave up..

after

$ hyperfine 'make phpstan'
Benchmark 1: make phpstan
  Time (mean ± σ):     48.054 s ±  0.398 s    [User: 326.638 s, System: 22.087 s]
  Range (min … max):   47.398 s … 48.890 s    10 runs

@ondrejmirtes
Copy link
Member

Yeah, that's a pretty old commit :) Thank you.

@ondrejmirtes ondrejmirtes merged commit 19d0669 into phpstan:1.9.x Feb 15, 2023
@herndlm herndlm deleted the remove-unneded-code branch February 15, 2023 21:45
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