[flang-commits] [flang] [flang] Catch NULL(MOLD=assumed-rank) (PR #95270)
via flang-commits
flang-commits at lists.llvm.org
Wed Jun 12 10:11:35 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-semantics
Author: Peter Klausler (klausler)
<details>
<summary>Changes</summary>
An assumed-rank dummy argument is not an acceptable MOLD argument to NULL(), whose result must have a known rank at compilation time.
---
Full diff: https://github.com/llvm/llvm-project/pull/95270.diff
2 Files Affected:
- (modified) flang/lib/Evaluate/intrinsics.cpp (+4)
- (modified) flang/test/Semantics/null01.f90 (+7-1)
``````````diff
diff --git a/flang/lib/Evaluate/intrinsics.cpp b/flang/lib/Evaluate/intrinsics.cpp
index 58c1b6989f495..ace316174a892 100644
--- a/flang/lib/Evaluate/intrinsics.cpp
+++ b/flang/lib/Evaluate/intrinsics.cpp
@@ -2691,6 +2691,10 @@ SpecificCall IntrinsicProcTable::Implementation::HandleNull(
mold = nullptr;
}
if (mold) {
+ if (IsAssumedRank(*arguments[0])) {
+ context.messages().Say(arguments[0]->sourceLocation(),
+ "MOLD= argument to NULL() must not be assumed-rank"_err_en_US);
+ }
bool isProcPtrTarget{
IsProcedurePointerTarget(*mold) && !IsNullObjectPointer(*mold)};
if (isProcPtrTarget || IsAllocatableOrPointerObject(*mold)) {
diff --git a/flang/test/Semantics/null01.f90 b/flang/test/Semantics/null01.f90
index 3bf620048e2f2..04d94865356b0 100644
--- a/flang/test/Semantics/null01.f90
+++ b/flang/test/Semantics/null01.f90
@@ -151,10 +151,16 @@ subroutine s1(x)
subroutine s2(x)
type(pdt(*)), pointer, intent(in) :: x
end
- subroutine test
+ subroutine s3(ar)
+ real, pointer :: ar(..)
+ end
+ subroutine test(ar)
+ real, pointer :: ar(..)
!ERROR: Actual argument associated with dummy argument 'x=' is a NULL() pointer without a MOLD= to provide a character length
call s1(null())
!ERROR: Actual argument associated with dummy argument 'x=' is a NULL() pointer without a MOLD= to provide a value for the assumed type parameter 'n'
call s2(null())
+ !ERROR: MOLD= argument to NULL() must not be assumed-rank
+ call s3(null(ar))
end
end
``````````
</details>
https://github.com/llvm/llvm-project/pull/95270
More information about the flang-commits
mailing list