[flang-commits] [flang] [flang][OpenMP] Fix crash when a sliced array is specified in a forall within a workshare construct (PR #170913)

[Carlos Seo via flang-commits]

ceseo wrote:

> Thank you for the bug fix, but I'm not sure this is right yet.
> 
> The problem you describe seems to indicate that it is not safe to parallelise when forall/where need to use the runtime stack (for example the call you mentioned). I'm not sure if removing nowait is the right fix.
> 
> Please could you add lit tests demonstrating the changes (perhaps based on the issue). Perhaps with some IR it would be clearer why nowait helps (and regression tests are good in themselves).

Any thoughts about this new patch?

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