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

via flang-commits flang-commits at lists.llvm.org
Sun Oct 4 06:48:01 PDT 2026


Author: Krzysztof Parzyszek
Date: 2026-10-04T08:47:55-05:00
New Revision: ed390ca4d61b412e595757b35dcb4fff07147c9e

URL: https://github.com/llvm/llvm-project/commit/ed390ca4d61b412e595757b35dcb4fff07147c9e
DIFF: https://github.com/llvm/llvm-project/commit/ed390ca4d61b412e595757b35dcb4fff07147c9e.diff

LOG: [flang][OpenMP] Remove unnecessary check for extension clauses (#228878)

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

Added: 
    

Modified: 
    flang/lib/Semantics/check-omp-syntax.cpp

Removed: 
    


################################################################################
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