Skip to content
This repository has been archived by the owner on Aug 15, 2024. It is now read-only.

perf/arc slice polys #42

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

Oppen
Copy link

@Oppen Oppen commented Apr 26, 2024

  • perf(experimental): use EcoVec for Polynomials
  • Hack around uninit
  • outer shouldn't be ecow and some clones shouldn't be owned
  • add sanitization
  • more sanitization
  • more sanitization
  • more sanitization
  • more sanitization
  • more sanitization
  • try to force unique
  • unsanitize
  • unsanitize
  • perf: use Arc<[P]> for Polynomial

What ❔

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.

This reverts commit 1998807.

Temporarily revert for testing, since this ensures I'm based on what era
is running.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant