[flang-commits] [flang] [flang][OpenMP] Recompute descriptor across workdistribute target fission (PR #225607)

[via flang-commits]

agozillon wrote:

Thank you very much, this all LGTM, the more expanded tests give me a good idea of the intent now :-) 

I would however, urge you not to let Claude write the check lines like it's done in workdistribute-saxpy-allocatable.f90 (and that I assume it done in the prior test). I've had it do the same thing (it really likes to cut corners), but when it's just a bunch of very ambiguous structure check lines it's incredibly hard to understand what the original intent of the test was, especially if you have no code in front of you to compare it against! This might just be a personal opinion though, as I hate when the AI writes tests like this and I have to berate it into writing something human readable/verifiable.

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