[llvm-branch-commits] [flang] [flang] Lower REDUCE intrinsic with DIM argument (PR #94771)

[Slava Zakharin via llvm-branch-commits]

Valentin Clement =?utf-8?b?KOODkOODrOODsw=?Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/94771 at github.com>


================
@@ -540,9 +606,31 @@ struct ForcedReduceComplex16 {
       auto strTy = fir::ReferenceType::get(mlir::IntegerType::get(ctx, 8));
       auto intTy = mlir::IntegerType::get(ctx, 8 * sizeof(int));
       auto refTy = fir::ReferenceType::get(ty);
+      auto refBoxTy = fir::ReferenceType::get(boxTy);
----------------
vzakhari wrote:

Hmm, I must be missing something.

The runtime API is defined as:
```
void RTDEF(CppReduceComplex16)(std::complex<CppFloat128Type> &result,
    const Descriptor &array,
    ReductionOperation<std::complex<CppFloat128Type>> operation,
    const char *source, int line, int dim, const Descriptor *mask,
    const std::complex<CppFloat128Type> *identity, bool ordered) {
```
So the first argument should be a reference to a complex value, isn't it right?

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