[Openmp-commits] [openmp] [OpenMP][NFC] Remove `DelayedBinDesc` (PR #74360)

Jon Chesterfield via Openmp-commits openmp-commits at lists.llvm.org
Tue Dec 5 08:43:12 PST 2023


JonChesterfield wrote:

It wasn't dead when we wrote it. It stopped HSA segfaulting when running any openmp programs.

The code might be dead now. I expect git log will find a "NFC" patch that changed the control flow so that this no guard no longer works, with no associated tests.

Naturally we don't test with dlopen'ed HSA on CI so when it stopped working we didn't notice.

As an argument against deleting dynamic_cuda and dynamic_hsa, instead of fixing them, those are the expected mechanisms by which `apt install llvm` might give users a working toolchain, since the distribution build machines generally won't have rocm or cuda installed to satisfy the link dependency.

https://github.com/llvm/llvm-project/pull/74360


More information about the Openmp-commits mailing list