Bugfix/rhornung67/fix sycl #112
Conversation
|
@adrienbernede This has been tested in RAJA and RAJAPerf and fixes issues in both with running SYCL CI job on corona in GitLab. After this is merged, this will need to be pulled in to RAJA develop. We can use this PR as a starting point: LLNL/RAJA#1758 Then, merging RAJA develop into this RAJAPerf PR LLNL/RAJAPerf#480 and we should be good. |
There was a problem hiding this comment.
My conclusion was that RAJAPerf needed both RAJA_ENABLE_SYCL and ENABLE_SYCL to work without change to the source.
I was trying to find a solution that does not affect the source so that we can build the last release of RAJAPerf with Spack. If your fix also modifies RAJAPerf build system, then Spack will only be able to build RAJAPerf @ develop.
|
ENABLE_SYCL is not needed and there are no changes to the RAJAPerf source or build system. The issue was the incorrect conditional in the RAJAPerf build_and_test.sh script. There is a RAJAPerf PR that has all GitLab CI working with this branch. When this is merged, other PRs will pull this into RAJA and RAJAPerf. |
This PR resolves issues related to RAJAPerf for running GitLab CI SYCL job on corona.