1
92%
master: 92%

Ran 13 Feb 2018 01:15AM UTC

Files 32

Run time 2s

Badge

Embed ▾

Committed 13 Feb 2018 12:53AM UTC coverage: 87.055% (+0.007%) from 87.048%

Job # COMPILER=g++ C_COMPILER=gcc BUILD_TYPE=Coverage

Build Type

Pull #527

travis-ci

Committed by

web-flow

Commit Message

Improve State packing: put important members on first cache line.

This patch does a few different things to ensure commonly accessed
data is on the first cache line of the `State` object.

First, it moves the `error_occurred_` member to reside after
the `started_` and `finished_` bools, since there was internal
padding there that was unused.

Second, it moves `batch_leftover_` and `max_iterations` further up
in the struct declaration. These variables are used in the calculation
of `iterations()` which users might call within the loop. Therefore
it's more important they exist on the first cache line.

Finally, this patch turns the bool members into bitfields. Although
this shouldn't have much of an effect currently, because padding is
still needed between the last bool and the first size_t, it should
help in future changes that require more "bool like" members.

Pull Request Pull Request #527: Improve State packing: put important members on first cache line.

Coverage Stats

1493 of 1715 relevant lines covered (87.06%)

4635615.22 hits per line

google / benchmark / 1349 / 1
92%
master: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1349.1 (COMPILER=g++ C_COMPILER=gcc BUILD_TYPE=Coverage)

google / benchmark / 1349 / 1 92% master: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1349.1 (COMPILER=g++ C_COMPILER=gcc BUILD_TYPE=Coverage)

google / benchmark / 1349 / 1
92%
master: 92%

README BADGES
x