Skip to content

sped up TestIO a bit by only processing code as C++ if necessary#4815

Merged
danmar merged 1 commit into
cppcheck-opensource:mainfrom
firewave:testio
Feb 23, 2023
Merged

sped up TestIO a bit by only processing code as C++ if necessary#4815
danmar merged 1 commit into
cppcheck-opensource:mainfrom
firewave:testio

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Feb 22, 2023

See https://trac.cppcheck.net/ticket/10663 about the actual culprit Library::detectContainer().

Clang 15 4,445,753,404 -> 3,551,472,880

before:

Benchmark 1: bin/testrunner TestIO
  Time (mean ± σ):      2.262 s ±  0.048 s    [User: 2.107 s, System: 0.154 s]
  Range (min … max):    2.205 s …  2.325 s    10 runs

after:

Benchmark 1: bin/testrunner TestIO
  Time (mean ± σ):      1.838 s ±  0.043 s    [User: 1.670 s, System: 0.165 s]
  Range (min … max):    1.773 s …  1.941 s    10 runs

@danmar danmar merged commit edc9cfb into cppcheck-opensource:main Feb 23, 2023
@firewave firewave deleted the testio branch February 23, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@firewave @danmar