[flang-commits] [flang] [llvm] [flang][OpenMP] Move clause validity checks into OpenMP-specific code (PR #205607)

[Sairudra More via flang-commits]

Saieiei wrote:

Nice cleanup. Non-blocking test-coverage thought: would it be worth adding an isolated begin/end uniqueness case for the new merged clause checking path?

I noticed duplicate `nowait` coverage where both occurrences are on the end directive, for example `!$omp end sections nowait nowait` / `!$omp end workshare nowait nowait`. A minimal case like `!$omp do nowait` paired with `!$omp end do nowait` would lock in the cross begin/end check directly.

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