[Mlir-commits] [flang] [mlir] [Flang][MLIR][OpenMP] - Add support for firstprivate when translating omp.target ops from MLIR to LLVMIR (PR #131213)

[Tom Eccles]

================
@@ -92,7 +103,7 @@ class MapsForPrivatizedSymbolsPass
         /*varPtrPtr=*/Value{},
         /*members=*/SmallVector<Value>{},
         /*member_index=*/mlir::ArrayAttr{},
-        /*bounds=*/ValueRange{},
+        /*bounds=*/boundsOps.empty() ? SmallVector<Value>{} : boundsOps,
----------------
tblah wrote:

This is unnecessary. An empty `boundsOps` is already an empty `SmallVector<Value>`.

https://github.com/llvm/llvm-project/pull/131213