https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/140971
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane updated
https://github.com/llvm/llvm-project/pull/140971
>From ec6359330368ed72d6e4354afdcf6d0a32f9a2f9 Mon Sep 17 00:00:00 2001
From: erichkeane
Date: Tue, 20 May 2025 08:55:39 -0700
Subject: [PATCH 1/2] [OpenACC][CIR] Add lowering for 'copy' array indexes
The ar
@@ -184,37 +189,94 @@ class OpenACCClauseCIREmitter final
mlir::Location beginLoc;
mlir::Value varValue;
llvm::StringRef name;
+llvm::SmallVector bounds;
};
+ mlir::Value createBound(mlir::Location boundLoc, mlir::Value lowerBound,
+
https://github.com/andykaylor approved this pull request.
lgtm with just one nit
https://github.com/llvm/llvm-project/pull/140971
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/andykaylor edited
https://github.com/llvm/llvm-project/pull/140971
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/razvanlupusoru approved this pull request.
Nice work!
https://github.com/llvm/llvm-project/pull/140971
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -184,37 +189,94 @@ class OpenACCClauseCIREmitter final
mlir::Location beginLoc;
mlir::Value varValue;
llvm::StringRef name;
+llvm::SmallVector bounds;
};
+ mlir::Value createBound(mlir::Location boundLoc, mlir::Value lowerBound,
+
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: Erich Keane (erichkeane)
Changes
The array indexes(and sections) are represented by the acc.bounds operation,
which this ensures we fill in properly. The lowerbound is required, so we
always get that.
The upperbound or extent is requi
https://github.com/erichkeane created
https://github.com/llvm/llvm-project/pull/140971
The array indexes(and sections) are represented by the acc.bounds operation,
which this ensures we fill in properly. The lowerbound is required, so we
always get that.
The upperbound or extent is required.
