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

Lazier sparse bit matrix #53629

Merged
merged 2 commits into from
Aug 26, 2018
Merged

Conversation

nnethercote
Copy link
Contributor

A small NLL win.

r? @nikomatsakis

The new names are clearer.
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 23, 2018
Currently when a row is instantiated in SparseBitMatrix, any missing
rows prior to it are also fully instantiated.

This patch changes things so that those prior rows are minimally
instantiated (with a `None`). This avoids a decent number of allocations
in NLL, speeding up several benchmarks by up to 0.5%.

The patch also removes two unused methods, `len()` and
`iter_enumerated()`.
@nnethercote nnethercote force-pushed the lazier-SparseBitMatrix branch from da8efe6 to 002f03b Compare August 23, 2018 08:42
@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Aug 23, 2018

📌 Commit 002f03b has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 23, 2018
bors added a commit that referenced this pull request Aug 26, 2018
…akis

Lazier sparse bit matrix

A small NLL win.

r? @nikomatsakis
@bors
Copy link
Contributor

bors commented Aug 26, 2018

⌛ Testing commit 002f03b with merge caed80b...

@bors
Copy link
Contributor

bors commented Aug 26, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing caed80b to master...

@bors bors merged commit 002f03b into rust-lang:master Aug 26, 2018
@nnethercote nnethercote deleted the lazier-SparseBitMatrix branch August 27, 2018 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants