Skip to content

Conversation

@Ritanya-B-Bharadwaj
Copy link
Contributor

This fixes the issue - #162243 caused by #158134

@llvmbot llvmbot added clang Clang issues not falling into any other category clang:frontend Language frontend issues, e.g. anything involving "Sema" clang:openmp OpenMP related changes to Clang labels Oct 29, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 29, 2025

@llvm/pr-subscribers-clang

Author: None (Ritanya-B-Bharadwaj)

Changes

This fixes the issue - #162243 caused by #158134


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

1 Files Affected:

  • (modified) clang/lib/Sema/SemaOpenMP.cpp (+1-1)
diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp index 0fa21e89b1236..06b87a4208702 100644 --- a/clang/lib/Sema/SemaOpenMP.cpp +++ b/clang/lib/Sema/SemaOpenMP.cpp @@ -3286,7 +3286,7 @@ SemaOpenMP::ActOnOpenMPGroupPrivateDirective(SourceLocation Loc, ArrayRef<Expr *> VarList) { if (!getLangOpts().OpenMP || getLangOpts().OpenMP < 60) { Diag(Loc, diag::err_omp_unexpected_directive) - << getOpenMPDirectiveName(OMPD_groupprivate, getLangOpts().OpenMP); + << 1 << getOpenMPDirectiveName(OMPD_groupprivate, getLangOpts().OpenMP); return nullptr; } if (OMPGroupPrivateDecl *D = CheckOMPGroupPrivateDecl(Loc, VarList)) { 
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:frontend Language frontend issues, e.g. anything involving "Sema" clang:openmp OpenMP related changes to Clang clang Clang issues not falling into any other category

2 participants