[flang-commits] [flang] [flang] Lower REDUCE intrinsic with no DIM argument and rank 1 (PR #94652)

via flang-commits flang-commits at lists.llvm.org
Thu Jun 6 14:08:00 PDT 2024


================
@@ -522,7 +522,7 @@ static constexpr IntrinsicHandler handlers[]{
        {"operation", asAddr},
        {"dim", asValue},
        {"mask", asBox, handleDynamicOptional},
-       {"identity", asValue},
+       {"identity", asAddr},
        {"ordered", asValue}}},
----------------
jeanPerier wrote:

Don't the identity and ordered value need "handleDynamicOptional" support too?
It looks legit to me to pass OPTIONAL to those arguments and expect them to not be dereferenced if absent.

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


More information about the flang-commits mailing list