Skip to content

Conversation

@AmrDeveloper
Copy link
Member

Fix enumeration value 'OMPFuseDirectiveClass' not handled in switch warning

@llvmbot llvmbot added clang Clang issues not falling into any other category ClangIR Anything related to the ClangIR project labels Sep 29, 2025
@llvmbot
Copy link
Member

llvmbot commented Sep 29, 2025

@llvm/pr-subscribers-clangir

@llvm/pr-subscribers-clang

Author: Amr Hesham (AmrDeveloper)

Changes

Fix enumeration value 'OMPFuseDirectiveClass' not handled in switch warning


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

1 Files Affected:

  • (modified) clang/lib/CIR/CodeGen/CIRGenStmt.cpp (+1)
diff --git a/clang/lib/CIR/CodeGen/CIRGenStmt.cpp b/clang/lib/CIR/CodeGen/CIRGenStmt.cpp
index e842892d085d2..5a3c4427c2f6f 100644
--- a/clang/lib/CIR/CodeGen/CIRGenStmt.cpp
+++ b/clang/lib/CIR/CodeGen/CIRGenStmt.cpp
@@ -279,6 +279,7 @@ mlir::LogicalResult CIRGenFunction::emitStmt(const Stmt *s,
   case Stmt::OMPGenericLoopDirectiveClass:
   case Stmt::OMPReverseDirectiveClass:
   case Stmt::OMPInterchangeDirectiveClass:
+  case Stmt::OMPFuseDirectiveClass:
   case Stmt::OMPAssumeDirectiveClass:
   case Stmt::OMPMaskedDirectiveClass:
   case Stmt::OMPStripeDirectiveClass:

Copy link
Contributor

@andykaylor andykaylor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@andykaylor
Copy link
Contributor

Already handled by #161278

@AmrDeveloper
Copy link
Member Author

Ops, I think next time I will ping @makslevental too, so we can prevent duplication 🤔

@makslevental
Copy link
Contributor

Ops, I think next time I will ping @makslevental too, so we can prevent duplication 🤔

lol you don't need to ping me - i don't work on CIR - i just fixed it because it was blocking me 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang Clang issues not falling into any other category ClangIR Anything related to the ClangIR project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants