[PATCH] D91159: [flang]Add Semantic Checks for Allocate Directive
Irina Dobrescu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 17 02:43:11 PST 2020
Rin added a comment.
In D91159#2397842 <https://reviews.llvm.org/D91159#2397842>, @kiranchandramohan wrote:
> I) Are you handling only a subset of the restrictions here? I see the following in,
>
> 1. General restrictions.
>
> -> allocate directives that appear in a target region must specify an allocator clause unless a requires directive with the dynamic_allocators clause is present in the same compilation unit.
>
> 2. Fortran specific restrictions.
>
> -> List items specified in the allocate directive must not have the ALLOCATABLE attribute unless the directive is associated with an allocate statement.
> -> List items specified in an allocate directive that is associated with an allocate statement must be variables that are allocated by the allocate statement.
> -> Multiple directives can only be associated with an allocate statement if list items are specified on each allocate directive.
> -> If a list item has the SAVE attribute, is a common block name, or is declared in the scope of a module, then only predefined memory allocator parameters can be used in the allocator clause.
> -> A type parameter inquiry cannot appear in an allocate directive.
>
> Since the requires directive is not there I am assuming that only a subset of the check in the general restrictions can be checked now.
> Please provide a clarification saying that only a subset of checks are implemented or implement the above checks.
>
> II) A few files have no newline at end of file.
I) So sorry about that. Don't know how I managed to miss all of those. I'll start working on them.
II) My editor seems to be doing that, I'll fix it with the next update.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91159/new/
https://reviews.llvm.org/D91159
More information about the llvm-commits
mailing list