Differential D24104

Replace std::tie with == in operator==()
ClosedPublic
Actions

Authored by dfaure on Sep 20 2019, 3:58 AM.

Details

Reviewers
mwolff
Commits
R45:b2371b6bcafc: Replace std::tie with == in operator==()
Summary

Before: 136M cycles, 165M instructions, 38.34 ms
After: 125M cycles, 161M instructions, 35 ms

Clearly this doesn't get inlined...
[gcc 7.4.1, RelWithDebInfo build]

Diff Detail

Branch
2019_09_remove_more_ties
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 16820
Build 16838: arc lint + arc unit
dfaure requested review of this revision.Sep 20 2019, 3:58 AM
dfaure created this revision.
dfaure added a reviewer: mwolff.Sep 22 2019, 12:24 AM
dfaure set the repository for this revision to R45 Heaptrack.
mwolff accepted this revision.Sep 24 2019, 9:06 AM

sad but OK :(

This revision is now accepted and ready to land.Sep 24 2019, 9:06 AM
dfaure closed this revision.Sep 24 2019, 12:21 PM

Revision Contents
Changeset List

PathPackages
Msrc/analyze/accumulatedtracedata.h (6 lines)

Diff 66521

View Options

src/analyze/accumulatedtracedata.h

Loading...