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

stats: add symbol table for future stat name encoding #3927

Merged
merged 41 commits into from
Aug 28, 2018
Merged
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
8de2463
WIP Working symbol table with manual reference counting
ambuc Jul 23, 2018
1c45948
Added tests, better error handling for null symbol lookups
ambuc Jul 23, 2018
a3df93b
Fix formatting
ambuc Jul 23, 2018
a78ea8d
Add common/pure header
ambuc Jul 23, 2018
59e8129
Add new StatNameImpl type for encapsulating the SymbolVec
ambuc Jul 24, 2018
475dcd3
Move the bulk of the logic to a .cc
ambuc Jul 24, 2018
9b3594b
Update tests
ambuc Jul 24, 2018
e7b619a
Add the SymbolTableImpl .cc
ambuc Jul 24, 2018
b3f1b7f
Mark internal string as const ref
ambuc Jul 25, 2018
bdc714c
More documentation around size()
ambuc Jul 25, 2018
0ded3e5
Decls before fields
ambuc Jul 25, 2018
b8b3f60
Hide toSymbols() fn from public API
ambuc Jul 25, 2018
ae650c3
Add proper test for death on the decode of an invalid symbol
ambuc Jul 25, 2018
14e8bb3
Added documentation for toSymbol() / fromSymbol()
ambuc Jul 25, 2018
86dc207
Add free pool for symbol re-use
ambuc Jul 25, 2018
cb6c754
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Jul 25, 2018
ff30317
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Jul 25, 2018
6e42c86
Various nits and docs fixes
ambuc Jul 26, 2018
f848464
Use std::stack<> for free pool
ambuc Jul 26, 2018
3687311
Optimized bimap using string_views
ambuc Jul 26, 2018
2772592
Add string_view return type for fromSymbol()
ambuc Jul 26, 2018
edb35e3
Added symbolic struct for SharedSymbol
ambuc Jul 26, 2018
c872073
SymbolTable::encode() now accepts a string_view
ambuc Jul 26, 2018
2f83e68
Simpler string construction in toSymbol()
ambuc Jul 26, 2018
85c5d8b
Use ASSERT() instead of RELEASE_ASSERT()
ambuc Jul 27, 2018
7f93df9
Assorted nits, todos, renaming
ambuc Jul 27, 2018
190da3d
Various const nits, stronger assertion against overflow
ambuc Jul 27, 2018
f2da353
Remove accidentally committed file
ambuc Jul 27, 2018
9e68dfd
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Jul 27, 2018
8e794d5
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Jul 31, 2018
bd47676
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 1, 2018
19c6e65
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 1, 2018
da2b7d7
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 1, 2018
9466aa3
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 7, 2018
86dab2c
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 14, 2018
208512d
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 14, 2018
27aa402
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 14, 2018
aaba51b
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 21, 2018
278f92f
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 24, 2018
ce87e7e
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 27, 2018
efd3c6f
Merge remote-tracking branch 'upstream/master' into symbol-table
ambuc Aug 27, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'upstream/master' into symbol-table
Signed-off-by: James Buckland <[email protected]>
ambuc committed Aug 21, 2018
commit aaba51b738ab4e9a5401afe027cfb3fb61e7e5b8

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.