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

Kareem Ergawy via flang-commits flang-commits at lists.llvm.org
Wed Apr 24 21:14:53 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;
----------------
ergawy wrote:

I am not sure about this change yet to be honest. In particular, I am testing the effect of changing this line to: `bool contiguous = variable.isSimplyContiguous() || variable.isArray();`.

@jeanPerier let me know if you think the above change is a bad idea for any reason.

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


More information about the flang-commits mailing list