[clang] [llvm] [openmp] [OpenMP][offload] Fix dynamic schedule tracking (PR #97065)

Gheorghe-Teodor Bercea via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 28 08:24:50 PDT 2024


doru1004 wrote:

> Could you provide a more descriptive summary?
> 
> I thought we discussed that the dynamic support would just use the static scheduler, but this seems to implement it? I personally don't want to see more things in the OpenMP runtime relying on `malloc` if we can avoid it.

I can add a summary.

The problem with the dispatch loop right now is that it is just not testable/runnable. This patch fixes that aspect of it. So wherever the dispatch loop is used it has to be correct.

We can re-evaluate the use of static schedule to implement dynamic schedule once this actually works.

Malloc cannot be helped here if we want to have correctness. Currently it is just broken and not even runnable.


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


More information about the cfe-commits mailing list