Skip to content

Conversation

gharlan
Copy link
Contributor

@gharlan gharlan commented Sep 10, 2025

@staabm
Copy link
Contributor

staabm commented Sep 11, 2025

Would be interesting whether a raw message based baseline is faster than a regular one, because we need less regex matching..?

Could you compare the runtime?

@ondrejmirtes
Copy link
Member

For huge baselines there has to be a difference for sure

@ondrejmirtes ondrejmirtes merged commit e46e694 into phpstan:2.1.x Sep 11, 2025
446 of 457 checks passed
@ondrejmirtes
Copy link
Member

Thank you!

@gharlan gharlan deleted the baseline-rawMessage branch September 11, 2025 08:16
@staabm
Copy link
Contributor

staabm commented Sep 11, 2025

Could you compare the runtime?

just ran time make phpstan before and after this PR on phpstan-src on macOS M4 Pro multiple times:

before: ~29-30s
after: ~27-28s

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.

3 participants