diff --git a/flang/include/flang/Semantics/symbol.h b/flang/include/flang/Semantics/symbol.h --- a/flang/include/flang/Semantics/symbol.h +++ b/flang/include/flang/Semantics/symbol.h @@ -639,6 +639,10 @@ [](const UseDetails &x) { return x.symbol().Rank(); }, [](const HostAssocDetails &x) { return x.symbol().Rank(); }, [](const ObjectEntityDetails &oed) { return oed.shape().Rank(); }, + [](const ProcEntityDetails &ped) { + const Symbol *iface{ped.interface().symbol()}; + return iface ? iface->Rank() : 0; + }, [](const AssocEntityDetails &aed) { if (const auto &expr{aed.expr()}) { if (auto assocRank{aed.rank()}) { diff --git a/flang/lib/Evaluate/shape.cpp b/flang/lib/Evaluate/shape.cpp --- a/flang/lib/Evaluate/shape.cpp +++ b/flang/lib/Evaluate/shape.cpp @@ -690,9 +690,9 @@ // to symbols that belong to the subroutine scope and are // meaningless on the caller side without the related call // expression. - for (auto extent : *resultShape) { - if (extent && !IsConstantExpr(*extent)) { - return std::nullopt; + for (auto &extent : *resultShape) { + if (extent && !IsActuallyConstant(*extent)) { + extent.reset(); } } } diff --git a/flang/test/Evaluate/rewrite01.f90 b/flang/test/Evaluate/rewrite01.f90 --- a/flang/test/Evaluate/rewrite01.f90 +++ b/flang/test/Evaluate/rewrite01.f90 @@ -57,6 +57,13 @@ end subroutine subroutine shape_test(x, n, m) + abstract interface + function foo(n) + integer, intent(in) :: n + real, pointer :: foo(:,:) + end function + end interface + procedure(foo), pointer :: pf integer :: x(n, m) !CHECK: PRINT *, [INTEGER(4)::int(size(x,dim=1,kind=8),kind=4),int(size(x,dim=2,kind=8),kind=4)] print *, shape(x) @@ -66,6 +73,8 @@ print *, shape(returns_array_2(m)) !CHECK: PRINT *, [INTEGER(8)::42_8] print *, shape(returns_array_3(), kind=8) + !CHECK: PRINT *, 2_4 + print *, rank(pf(1)) end subroutine subroutine lbound_test(x, n, m)