[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-30 Thread Mariya Podchishchaeva via cfe-commits
Fznamznon wrote: > LLVM Buildbot has detected a new failure on builder > openmp-offload-amdgpu-runtime running on omp-vega20-0 while building clang at > step 7 "Add check check-offload". That seems to be failing the same way for some other PRs. https://github.com/llvm/llvm-project/pull/110243

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-30 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-offload-amdgpu-runtime` running on `omp-vega20-0` while building `clang` at step 7 "Add check check-offload". Full details are available at: https://lab.llvm.org/buildbot/#/builders/30/builds/7308 Here is the relevan

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-30 Thread Mariya Podchishchaeva via cfe-commits
https://github.com/Fznamznon closed https://github.com/llvm/llvm-project/pull/110243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-30 Thread Mariya Podchishchaeva via cfe-commits
https://github.com/Fznamznon updated https://github.com/llvm/llvm-project/pull/110243 >From 314a8f4d4f8e003881bed0c320f9ee80ba0b8f8c Mon Sep 17 00:00:00 2001 From: "Podchishchaeva, Mariya" Date: Fri, 27 Sep 2024 04:00:31 -0700 Subject: [PATCH] [clang] Fix static analyzer concerns It seems in c

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread Alexey Bataev via cfe-commits
@@ -9195,7 +9195,7 @@ static bool checkOpenMPIterationSpace( SemaRef.Diag(CollapseLoopCountExpr->getExprLoc(), diag::note_omp_collapse_ordered_expr) << 0 << CollapseLoopCountExpr->getSourceRange(); - else + else if (OrderedLoop

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread Alexey Bataev via cfe-commits
@@ -9195,7 +9195,7 @@ static bool checkOpenMPIterationSpace( SemaRef.Diag(CollapseLoopCountExpr->getExprLoc(), diag::note_omp_collapse_ordered_expr) << 0 << CollapseLoopCountExpr->getSourceRange(); - else + else if (OrderedLoop

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread Mariya Podchishchaeva via cfe-commits
https://github.com/Fznamznon updated https://github.com/llvm/llvm-project/pull/110243 >From 314a8f4d4f8e003881bed0c320f9ee80ba0b8f8c Mon Sep 17 00:00:00 2001 From: "Podchishchaeva, Mariya" Date: Fri, 27 Sep 2024 04:00:31 -0700 Subject: [PATCH] [clang] Fix static analyzer concerns It seems in c

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread Alexey Bataev via cfe-commits
https://github.com/alexey-bataev approved this pull request. https://github.com/llvm/llvm-project/pull/110243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread Mariya Podchishchaeva via cfe-commits
@@ -9195,7 +9195,7 @@ static bool checkOpenMPIterationSpace( SemaRef.Diag(CollapseLoopCountExpr->getExprLoc(), diag::note_omp_collapse_ordered_expr) << 0 << CollapseLoopCountExpr->getSourceRange(); - else + else if (OrderedLoop

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Mariya Podchishchaeva (Fznamznon) Changes It seems in checkOpenMPIterationSpace `OrderedLoopCountExpr` can also be null, so check before dereferencing. --- Full diff: https://github.com/llvm/llvm-project/pull/110243.diff 1 Files Affecte

[clang] [clang] Fix static analyzer concerns (PR #110243)

2024-09-27 Thread Mariya Podchishchaeva via cfe-commits
https://github.com/Fznamznon created https://github.com/llvm/llvm-project/pull/110243 It seems in checkOpenMPIterationSpace `OrderedLoopCountExpr` can also be null, so check before dereferencing. >From 314a8f4d4f8e003881bed0c320f9ee80ba0b8f8c Mon Sep 17 00:00:00 2001 From: "Podchishchaeva, Mar