[clang] [OpenMP][NFC] Fix argument order of SourceLocations for allocate clause (PR #94777)

Mike Rice via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 7 10:53:54 PDT 2024


https://github.com/mikerice1969 created https://github.com/llvm/llvm-project/pull/94777

Static verifier caught passing ColonLoc/LParenLoc in wrong order. Marked as NFC since these don't seem to be used for anything currently that I can think to test for.

>From ae0438eda2b5fcffc70b991d2c496bac9b89a5a2 Mon Sep 17 00:00:00 2001
From: Mike Rice <michael.p.rice at intel.com>
Date: Fri, 7 Jun 2024 10:47:55 -0700
Subject: [PATCH] [OpenMP][NFC] Fix argument order of SourceLocations for
 allocate clause

Static verifier caught passing ColonLoc/LParenLoc in wrong order.
Marked as NFC since these don't seem to be used for anything currently
that I can think to test for.
---
 clang/lib/Sema/SemaOpenMP.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp
index 6e6815328e913..f1ebe9d3b9db7 100644
--- a/clang/lib/Sema/SemaOpenMP.cpp
+++ b/clang/lib/Sema/SemaOpenMP.cpp
@@ -24331,7 +24331,7 @@ SemaOpenMP::ActOnOpenMPHasDeviceAddrClause(ArrayRef<Expr *> VarList,
 
 OMPClause *SemaOpenMP::ActOnOpenMPAllocateClause(
     Expr *Allocator, ArrayRef<Expr *> VarList, SourceLocation StartLoc,
-    SourceLocation ColonLoc, SourceLocation LParenLoc, SourceLocation EndLoc) {
+    SourceLocation LParenLoc, SourceLocation ColonLoc, SourceLocation EndLoc) {
   if (Allocator) {
     // OpenMP [2.11.4 allocate Clause, Description]
     // allocator is an expression of omp_allocator_handle_t type.



More information about the cfe-commits mailing list