Robert Chen

Closing this issue because it is fixed here https://github.com/LLNL/RAJA/pull/1339.

Notes to self: - Potential workaround to current SYCL compiler build issues with HIP https://github.com/intel/llvm/issues/11873 - Need at least rocm/5.6.0 due to SYCL's usage of specific API calls (e.g. `hipArray3DGetDescriptor`)

Preserving the instructions from 2022 for posterity - Instructions for building a working SYCL compiler on Corona: 1. module load rocm/5.1.0 2. module load gcc-tce/10.2.1 3. module load mvapich2-tce/2.3.6 4....

> Can we use a newer version of ROCm? 5.1 is old and we'll soon be moving to 6.0. Does 5.6 work? Yes, if you look at the comment on...

Hello @ajkunen, were the vector `min_n()` and `max_n()` functions meant to be run from the host? E.g. https://github.com/LLNL/RAJA/pull/1289/files#diff-86f4ee9832658f6f7d056f45bb84ef9d4e11673334eb81e7e8ebd9203b1de27dR38 When I try them on the device, the test mysteriously dies.

@rhornung67 @davidbeckingsale @MrBurmark @artv3 @ajkunen This should be ready to go again. Please take a look when you have a chance. Thanks!

Mike Collette recommended adding the `-fopenmp-version=45` flag to the link line, and it resolved the error. I recall needing to do this for Fortran+CUDA compilation, but I didn't think we'd...

> @rchen20 if the issue is resolved with the link line flag, please put up a PR with that change applied to the appropriate build script and/or host-config file. The...

Strange, I get the same error with non-IBM clang, and adding the flag solves the problem but begets another set of template errors: ``` [ 1%] Building CXX object blt/thirdparty_builtin/googletest/googletest/CMakeFiles/gtest.dir/src/gtest-all.cc.o...

After trying again with the latest RAJA/develop, I'm getting the same error linkage specification error as before, with the `-fopenmp-version=45` being a successful hack to get around this. I think...