Skip to content

Increase LLVM inline-threshold to 5000 - #130

Open
peterzhu2118 wants to merge 1 commit into
mainfrom
inline-threshold
Open

peterzhu2118 wants to merge 1 commit into
mainfrom
inline-threshold

Conversation

@peterzhu2118

Copy link
Copy Markdown
Member

Compared to the default of 225, this gives noticeable performance improvements.

For the following benchmark:

objs = []
2_000_000.times { objs << Object.new }

We can see this branch is about 10% faster:

Benchmark 1: branch
  Time (mean ± σ):     193.0 ms ±   3.3 ms    [User: 207.0 ms, System: 79.2 ms]
  Range (min … max):   189.1 ms … 203.1 ms    15 runs

Benchmark 2: master
  Time (mean ± σ):     214.9 ms ±   4.9 ms    [User: 245.2 ms, System: 79.5 ms]
  Range (min … max):   206.4 ms … 220.4 ms    13 runs

Summary
  branch ran
    1.11 ± 0.03 times faster than master

Compared to the default of 225, this gives noticeable performance
improvements.

For the following benchmark:

    objs = []
    2_000_000.times { objs << Object.new }

We can see this branch is about 10% faster:

    Benchmark 1: branch
      Time (mean ± σ):     193.0 ms ±   3.3 ms    [User: 207.0 ms, System: 79.2 ms]
      Range (min … max):   189.1 ms … 203.1 ms    15 runs

    Benchmark 2: master
      Time (mean ± σ):     214.9 ms ±   4.9 ms    [User: 245.2 ms, System: 79.5 ms]
      Range (min … max):   206.4 ms … 220.4 ms    13 runs

    Summary
      branch ran
        1.11 ± 0.03 times faster than master
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.

1 participant