[llvm-branch-commits] [flang] [flang][OpenMP] Switch clause verification to descriptor-based (PR #229527)

[Sairudra More via llvm-branch-commits]

================
@@ -2352,8 +2139,6 @@ void OmpStructureChecker::Enter(const parser::OpenMPDepobjConstruct &x) {
     }
   }
   if (clauses.v.size() != 1) {
-    context_.Say(
-        x.source, "The DEPOBJ construct requires a single clause"_err_en_US);
----------------
Saieiei wrote:

Without this error, `!$omp depobj(d) update(in) destroy` with `-fopenmp-version=60` reaches the `clauses.size() == 1` assertion in lowering, although 6.0 section 17.9.3 still allows only one clause with the argument form. Keeping this error for that form at 6.0 fixes it for me.

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