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

Add CompoundAdder component #98

Merged
merged 36 commits into from
Oct 11, 2024
Merged

Add CompoundAdder component #98

merged 36 commits into from
Oct 11, 2024

Conversation

aasorokiin
Copy link
Contributor

Add CompoundAdder component.
Integer adder with sum+1 output.
MockCompoundAdder: Trivial implementation with sum and sum+1
CarrySelectCompoundAdder: Carry-select adder with double carry path. One path is for sum and another is for sum+1.

carry-ripple block size selection algorithm:
splitSelectAdderAlgorithmSingleBlock - only one block is generated
splitSelectAdderAlgorithm4Bit - 4-bit ripple-carry adder blocks are used.

Copy link
Contributor

@mkorbel1 mkorbel1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

lib/src/arithmetic/compound_adder.dart Outdated Show resolved Hide resolved
lib/src/arithmetic/compound_adder.dart Outdated Show resolved Hide resolved
lib/src/arithmetic/ripple_carry_adder.dart Outdated Show resolved Hide resolved
lib/src/arithmetic/compound_adder.dart Outdated Show resolved Hide resolved
lib/src/arithmetic/compound_adder.dart Show resolved Hide resolved
lib/src/arithmetic/compound_adder.dart Show resolved Hide resolved
test/arithmetic/compound_adder_test.dart Outdated Show resolved Hide resolved
anton.a.sorokin and others added 2 commits October 2, 2024 18:41
… Remove ripple-carry adder with required carry input.
Add compound adder doc. Make ripple-carry adder carry input optional.…
Copy link
Contributor

@mkorbel1 mkorbel1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes so far look good!

anton.a.sorokin and others added 2 commits October 4, 2024 10:30
anton.a.sorokin and others added 2 commits October 8, 2024 16:44
Copy link
Contributor

@mkorbel1 mkorbel1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@mkorbel1 mkorbel1 merged commit 68eb1ad into intel:main Oct 11, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants