[clang][OpenMP] Fix directive in ActOnOpenMPTargetParallelForSimdDire…

[kparzysz]

…ctive

The function ActOnOpenMPTargetParallelForSimdDirective gets the number of capture levels for OMPD_target_parallel_for, whereas the intended directive is OMPD_target_parallel_for_simd.

[kparzysz]

I suspect this was a mistake, let me know if you intended to use the existing directive here.

[llvmbot]

@llvm/pr-subscribers-clang

Author: Krzysztof Parzyszek (kparzysz)

Changes

…ctive

The function ActOnOpenMPTargetParallelForSimdDirective gets the number of capture levels for OMPD_target_parallel_for, whereas the intended directive is OMPD_target_parallel_for_simd.

---

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

1 Files Affected:

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

diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp
index 0cc0cbacb37548..e9ad7bbde0f9b5 100644
--- a/clang/lib/Sema/SemaOpenMP.cpp
+++ b/clang/lib/Sema/SemaOpenMP.cpp
@@ -14366,7 +14366,8 @@ StmtResult Sema::ActOnOpenMPTargetParallelForSimdDirective(
   // The point of exit cannot be a branch out of the structured block.
   // longjmp() and throw() must not violate the entry/exit criteria.
   CS->getCapturedDecl()->setNothrow();
-  for (int ThisCaptureLevel = getOpenMPCaptureLevels(OMPD_target_parallel_for);
+  for (int ThisCaptureLevel =
+           getOpenMPCaptureLevels(OMPD_target_parallel_for_simd);
        ThisCaptureLevel > 1; --ThisCaptureLevel) {
     CS = cast<CapturedStmt>(CS->getCapturedStmt());
     // 1.2.2 OpenMP Language Terminology

[alexey-bataev]

LG.
In the current implementation there is just no difference, they have same number of capture levels