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

CI-unixish.yml: show check time for each file in selfcheck #6618

Merged
merged 1 commit into from
Jul 21, 2024

Conversation

firewave
Copy link
Collaborator

@firewave firewave commented Jul 20, 2024

The existing usage of --showtime=top5_summary is broken and also would not provide any context of which file the data belongs to.

@firewave
Copy link
Collaborator Author

If we were using a project this information could be used to order the files to process the slowest at first.

@firewave
Copy link
Collaborator Author

Files > 60 seconds:

Check time: lib/astutils.cpp: 87.222s
Check time: lib/checkclass.cpp: 106.768s
Check time: lib/checkother.cpp: 86.4929s
Check time: lib/checkstl.cpp: 84.4599s
Check time: lib/checkuninitvar.cpp: 72.8557s
Check time: lib/checkunusedvar.cpp: 88.7682s
Check time: lib/checkio.cpp: 322.58s
Check time: lib/templatesimplifier.cpp: 107.094s
Check time: lib/symboldatabase.cpp: 421.475s
Check time: lib/tokenize.cpp: 544.499s
Check time: cli/cmdlineparser.cpp: 1546.02s

@firewave
Copy link
Collaborator Author

If we were using a project this information could be used to order the files to process the slowest at first.

Won't actually help since cli/cmdlineparser.cpp is by far the longest running file and it is close to be one of the first being analyzed.

@firewave firewave merged commit cda7453 into danmar:main Jul 21, 2024
63 checks passed
@firewave firewave deleted the selfcheck-time branch July 21, 2024 10:41
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