Skip to content

Conversation

@PaulMullowney
Copy link

This is a version of the driver I wrote which I've been using for validation. Ultimately, we may not want to keep this but it may be useful for a while as we attempt to make the OpenMP version portable.

@PaulMullowney PaulMullowney force-pushed the new_omp_driver branch 2 times, most recently from 5f36177 to 9115c0f Compare October 3, 2025 15:10
Copy link
Collaborator

@reuterbal reuterbal left a comment

Choose a reason for hiding this comment

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

No objections to keeping this around for a while. Am I understanding correctly that this is a kind of halfway-house between the ecrad_ifs and ecrad_ifs_blocked drivers, with blocked allocations but avoiding the slices out of a single allocation?

Ideally, we should add this driver to the Ctest definitions so tests are being run for it, too. Could you add it to the list of drivers in these two places?

  1. For running the binary:
    foreach( binary ecrad_${PREC} ecrad_ifs_${PREC} ecrad_ifs_blocked_${PREC} )
  2. For validating the results
    foreach( binary ecrad_ifs_${PREC} ecrad_ifs_blocked_${PREC} )

@PaulMullowney PaulMullowney deleted the new_omp_driver branch October 15, 2025 15:23
@PaulMullowney PaulMullowney restored the new_omp_driver branch October 15, 2025 15:29
@PaulMullowney PaulMullowney reopened this Oct 15, 2025
…this version but it will be use to have around for a while.
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.

2 participants