[Openmp-commits] [openmp] [OpenMP] Fix Taskgraph bugs (PR #136837)

[Josep Pinot via Openmp-commits]

jpinot wrote:

> As these changes correct issues with dynamic arrays in the task graph structure, should we also add tests regarding the size and width of the task graph?

I added a new [test](https://github.com/llvm/llvm-project/pull/136837/commits/2c7db5f7f50913d5b341b577e7f037484b05cda0) that focuses on TDG with dependencies and multiple successors. While there is already a test for TDG with dependencies ([omp_record_replay_deps.cpp](https://github.com/llvm/llvm-project/blob/main/openmp/runtime/test/tasking/omp_record_replay_deps.cpp)), it doesn’t exercise the expansion of the `successors` array directly.

Let me know you thoughts on whether keeping this as a new targeted test, or if it would be better to update the existing test to incorporate these scenarios, even if it adds some complexity.


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