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

Krzysztof Parzyszek via flang-commits flang-commits at lists.llvm.org
Sun Oct 4 05:31:58 PDT 2026


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

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

>From 0ef3ccd0680e0e00fc9ab810eddab876b46b8792 Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek <Krzysztof.Parzyszek at amd.com>
Date: Sun, 4 Oct 2026 07:26:27 -0500
Subject: [PATCH] [flang][OpenMP] Remove unnecessary check for extension
 clauses

Remove the check for extension clauses. These now do have descriptors,
so the check guarding the descriptor retrieval is no longer necessary.
---
 flang/lib/Semantics/check-omp-syntax.cpp | 4 ----
 1 file changed, 4 deletions(-)

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



More information about the flang-commits mailing list