[flang-commits] [flang] [flang][OpenMP] Remove unnecessary check for extension clauses (PR #228878)

[via flang-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-flang-semantics

Author: Krzysztof Parzyszek (kparzysz)

<details>
<summary>Changes</summary>

Remove the check for extension clauses. These now do have descriptors, so the check guarding the descriptor retrieval is no longer necessary.

---
Full diff: https://github.com/llvm/llvm-project/pull/228878.diff


1 Files Affected:

- (modified) flang/lib/Semantics/check-omp-syntax.cpp (-4) 


``````````diff
diff --git a/flang/lib/Semantics/check-omp-syntax.cpp b/flang/lib/Semantics/check-omp-syntax.cpp
index ceda90edaf640..03f8e554be350 100644
--- a/flang/lib/Semantics/check-omp-syntax.cpp
+++ b/flang/lib/Semantics/check-omp-syntax.cpp
@@ -519,10 +519,6 @@ void OmpStructureChecker::VerifyModifierSyntax(const parser::OmpClause &x) {
   llvm::omp::Clause id{x.Id()};
   auto clauseId{WithSource(id, x.source)};
   switch (id) {
-  case llvm::omp::Clause::OMPC_ompx_bare:
-  case llvm::omp::Clause::OMPC_cancellation_construct_type:
-    // Those are extensions/synthetic clauses and they don't have descriptors.
-    break;
   case llvm::omp::Clause::OMPC_uses_allocators: {
     // The traits of the deprecated syntax are stored as a traits-array
     // modifier, but they are not the 5.2 modifier, so they must not be

``````````

</details>


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