[Clang][Sema] Remove duplicate check in if-condition

[DeinAlptraum]

Resolves #101041

[llvmbot]

@llvm/pr-subscribers-clang

Author: Jannick Kremer (DeinAlptraum)

Changes

Resolves #101041

---

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

1 Files Affected:

• (modified) clang/lib/Sema/SemaOpenMP.cpp (+1-2)

diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp
index 9c80b3eec914c..1d378e6b830ae 100644
--- a/clang/lib/Sema/SemaOpenMP.cpp
+++ b/clang/lib/Sema/SemaOpenMP.cpp
@@ -23087,8 +23087,7 @@ OMPClause *SemaOpenMP::ActOnOpenMPDoacrossClause(
   if (DSAStack->getCurrentDirective() == OMPD_ordered &&
       DepType != OMPC_DOACROSS_source && DepType != OMPC_DOACROSS_sink &&
       DepType != OMPC_DOACROSS_sink_omp_cur_iteration &&
-      DepType != OMPC_DOACROSS_source_omp_cur_iteration &&
-      DepType != OMPC_DOACROSS_source) {
+      DepType != OMPC_DOACROSS_source_omp_cur_iteration) {
     Diag(DepLoc, diag::err_omp_unexpected_clause_value)
         << "'source' or 'sink'" << getOpenMPClauseName(OMPC_doacross);
     return nullptr;

[DeinAlptraum]

@zyn0217 I hope you're the right person to ask for a review here?

[zyn0217]

Thanks.
(However I'm not an OpenMP maintainer, but since this PR is clearly an NFC change, I'd boldly approve it.)

[zyn0217]

(I presume you don't have commit access, so I'll help you merge it. Next time, please add an "NFC" term to the commit title if the PR doesn't contain any functional changes.)