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

tests: small cleanup #2320

Merged
merged 7 commits into from
Feb 13, 2022
Merged

tests: small cleanup #2320

merged 7 commits into from
Feb 13, 2022

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Feb 13, 2022

Add a test modulating DDM seed in analytical mode.
Use helper function to select the right execution mode.
Small style cleanups.

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

Pointed out by linter.

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

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
… way of selecting execution mode

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
The operations are fast so avoiding int->float conversion saves ~15% of
time.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
@jvesely jvesely added compiler Runtime Compiler tests Test changes labels Feb 13, 2022
@jvesely jvesely added this to In progress in LLVM Runtime Compiler via automation Feb 13, 2022
@github-actions
Copy link

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 merged commit 779d3c1 into PrincetonUniversity:devel Feb 13, 2022
LLVM Runtime Compiler automation moved this from In progress to Done Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler Runtime Compiler tests Test changes
Projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant