[flang-commits] [flang] [flang][MLIR][OpenMP] Extend delayed privatization for arrays and characters (PR #85023)

via flang-commits flang-commits at lists.llvm.org
Fri May 3 07:23:34 PDT 2024


================
@@ -860,7 +859,9 @@ translateVariableToExtendedValue(mlir::Location loc, fir::FirOpBuilder &builder,
                                 fir::MutableProperties{});
 
   if (base.getType().isa<fir::BaseBoxType>()) {
-    if (!variable.isSimplyContiguous() || variable.isPolymorphic() ||
+
+    bool contiguous = variable.isSimplyContiguous() || contiguousHint;
----------------
jeanPerier wrote:

variable.isArray() would not be correct since this would be true for any array, including `x(:)` that is not contiguous. Your change using the extra knowledge you have from the symbol/original box makes sense to me.

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


More information about the flang-commits mailing list