[flang-commits] [PATCH] D130386: [flang] Allow restricted specific intrinsic functions as implicitly-interfaced procedure pointer targets

Wed Aug 17 10:14:08 PDT 2022

klausler added a comment.

In D130386#3728999 <https://reviews.llvm.org/D130386#3728999>, @ekieri wrote:

> In D130386#3727156 <https://reviews.llvm.org/D130386#3727156>, @klausler wrote:
>
>> In D130386#3726982 <https://reviews.llvm.org/D130386#3726982>, @ekieri wrote:
>>
>>> I am sorry the warning patch (D125804 <https://reviews.llvm.org/D125804>) is taking so long, I have not found much time to work with Flang lately. This patch causes a mismatch between emitted and expected warnings in associated.f90, line 141. Could you please have a look to see if the changed warning message is deliberate or not?
>>
>> A warning is appropriate there.
>
> Agree. Previously, the warning was about the dummy variable having incompatible attributes (intent(in) and value, respectively) for pointer and target. Now it is about the procedure being elemental when the pointer is not. Can you confirm this change is desired? Sorry if I was unclear.

Yes, the current warning is correct.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130386/new/

https://reviews.llvm.org/D130386