Skip to content

Cleaning up IDETabulationProblem#653

Merged
fabianbs96 merged 9 commits intodevelopmentfrom
f-DefaultAllTopFn
Aug 28, 2023
Merged

Cleaning up IDETabulationProblem#653
fabianbs96 merged 9 commits intodevelopmentfrom
f-DefaultAllTopFn

Conversation

@fabianbs96
Copy link
Member

This PR consists of the following cleanup tasks:

  • allTopFunction() now has a proper default
  • the flow-function templates (identityFlow(), lambdaFlow(), etc.) have been moved to their own class not requiring them to template on d_t and container_type anymore. This is one step towards easy switching between different container_types in order to find out, which one performs best for a given data-flow problem

@fabianbs96 fabianbs96 self-assigned this Aug 2, 2023
@fabianbs96 fabianbs96 marked this pull request as ready for review August 15, 2023 17:36
Copy link
Collaborator

@vulder vulder left a comment

Choose a reason for hiding this comment

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

For my parts, this looks fine

@fabianbs96 fabianbs96 merged commit b763b5f into development Aug 28, 2023
@fabianbs96 fabianbs96 deleted the f-DefaultAllTopFn branch August 28, 2023 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants