[flang-commits] [flang] [Flang][OpenMP] Correct ArrayElements in Reduction Clause (PR #196094)

[Tom Eccles via flang-commits]

================
@@ -75,6 +75,20 @@ static void processHostEvalClauses(lower::AbstractConverter &converter,
                                    mlir::Location loc);
 
 namespace {
+static bool isReductionObjectExpression(const Object *object) {
----------------
tblah wrote:

nit: needs a better name e.g. `isArrayElementExpression`.

Or if you intentionally designed this to allow extending to other kinds of reduction objects in the future, please explain with a comment. Right now I find this difficult to read. Maybe 'isExpressionLoweredAsReductionObject`

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