Skip to content

Refactor Cell and avoid overlapping mutable borrows. - #303

Closed
icmccorm wants to merge 3 commits into
supranational:masterfrom
icmccorm:patch
Closed

Refactor Cell and avoid overlapping mutable borrows.#303
icmccorm wants to merge 3 commits into
supranational:masterfrom
icmccorm:patch

Conversation

@icmccorm

@icmccorm icmccorm commented Oct 22, 2026

Copy link
Copy Markdown

This is an incomplete fix for #302. It covers the first two bugs listed, but not the third one, which may need a change to how threadpool is used.

@dot-asm dot-asm mentioned this pull request Oct 23, 2026
Closed
dot-asm added a commit that referenced this pull request Nov 6, 2026
@dot-asm
bindings/rust/src/pippenger.rs: resolve BorrowSanitizer misgivings.
cf7f2e3 
Concern is that the compiler may generate code prone to out-of-order loads
and stores. However, the allegedly susceptible operations are enclosed by
atomic operations that the compiler is not allowed to reorder, which
effectively guarantees the proper ordering.

Fixes #302, #303.
@dot-asm dot-asm closed this Nov 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@icmccorm @dot-asm