[flang-commits] [PATCH] D122899: [flang] add evaluate::IsAllocatable helper

[Peter Klausler via Phabricator via flang-commits]

klausler added a comment.

In D122899#3423067 <https://reviews.llvm.org/D122899#3423067>, @jeanPerier wrote:

> In D122899#3422673 <https://reviews.llvm.org/D122899#3422673>, @klausler wrote:
>
>> There is a naming clash with `semantics::IsAllocatable()`.
>
> I see. Would `evaluate::IsAllocatableEntity()` sounds good to you ?

"Designator" seems best.  "Variable" would also work, but the only variables that aren't designators are the returned values of pointer-valued functions, and they cannot be allocatable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122899