LLNL / axom

CS infrastructure components for HPC applications
BSD 3-Clause "New" or "Revised" License
159 stars 27 forks source link

Remaining Issues with HIP #787

Open bmhan12 opened 2 years ago

bmhan12 commented 2 years ago

Tasks from #778 that still remain:

EDIT:

EDIT x2:

EDIT x3:

rhornung67 commented 2 years ago

@bmhan12 you are using the latest BLT develop with amdclang++? what is being built when the -lhip_runtime error shows up?

bmhan12 commented 2 years ago

@rhornung67 With the latest blt develop and hipcc, I've seen the error appear with any CMake target that depends on umpire or RAJA (e.g. "umpire_smoke_test").

white238 commented 2 years ago

@rhornung67 With the latest blt develop and hipcc, I've seen it appear with any CMake target that depends on umpire or RAJA (e.g. "umpire_smoke_test").

This is more than likely because the Umpire release we are using is not exporting their hip_runtime target properly. This has been probably fixed in the new Umpire release. After both RAJA and Umpire have ironed out their BLT HIP support, I will help you with this.

We could also fake it. This makes a noop fake target in CMake:

add_library(hip_runtime INTERFACE)

Just stick it at the base of Axom somewhere with a giant TODO

rhornung67 commented 2 years ago

Actually, I ran across that and the issue was some "very clever" stuff in camp. Here is the camp branch I am using to test the new BLT in RAJA: https://github.com/LLNL/camp/tree/raja-blt-update This isn't a complete fix and I am waiting on a response from Tom for something better. @bmhan12 you may want to try that camp branch and this RAJA branch that I have working now: https://github.com/LLNL/RAJA/tree/blt-update. Umpire develop has the new BLT stuff in it. Umpire develop and the camp and RAJA branches I linked here are all using the same commit of BLT develop.

white238 commented 2 years ago

That makes more sense. Umpire is usually very good about exporting their targets properly.

rhornung67 commented 2 years ago

camp's cmake stuff is too fancy(!) for me....

kennyweiss commented 2 months ago

Bump. Is this still an issue? It might be worth reinvestigating the open tasks on this issue