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

llvm, OptimizationControlMechanism: Reuse costs computed by TransferWithCosts #2761

Merged
merged 6 commits into from
Aug 3, 2023

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Aug 3, 2023

Execute OCM ports when setting an allocation.
Reuse calculated costs from TransferWithCosts.
Drop "allocation" parameter to net_outcome.
Consolidate model_based_ocm tests.

…rt" in warning message

Allocation samples are applied to output ports.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
…dices

Remove duplicate index query.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
…igning control allocation

Matches Python semantics.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
… saved values

This allows to call the benchmark fixtures unconditionally.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
…ferWithCosts when computing net_outcome

Drop allocation vector structure from net_outcome argument list.
Enable tests of different cost combinations.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
The tests were almost identical but for the setting of the controller.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
@jvesely jvesely added the compiler Runtime Compiler label Aug 3, 2023
@github-actions
Copy link

github-actions bot commented Aug 3, 2023

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

@jvesely jvesely added this to the Compilation Parity milestone Aug 3, 2023
@jvesely jvesely merged commit 84b9a72 into PrincetonUniversity:devel Aug 3, 2023
31 checks passed
@jvesely jvesely deleted the llvm branch August 3, 2023 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler Runtime Compiler
Projects
Development

Successfully merging this pull request may close these issues.

1 participant