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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to