maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[flang] Address comments 

Original-commit: flang-compiler/f18@2a6473ce6a
This commit is contained in:
Jean Perier 2019-05-03 01:31:48 -07:00
parent c67710e5ae
commit 6370087abc
2 changed files with 28 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
flang/lib/semantics/check-allocate.cc
View File

@ -48,24 +48,11 @@ public:
allocateShapeSpecRank_{ShapeSpecRank(alloc)},
rank_{name_.symbol ? name_.symbol->Rank() : 0},
allocateCoarraySpecRank_{CoarraySpecRank(alloc)},
corank_{name_.symbol ? name_.symbol->Corank() : 0} {
GatherAllocationBasicInfo();
}
corank_{name_.symbol ? name_.symbol->Corank() : 0} {}
bool RunChecks(SemanticsContext &context);
private:
AllocateCheckerInfo &allocateInfo_;
const parser::AllocateObject &allocateObject_;
const parser::Name &name_;
const DeclTypeSpec *type_;
const int allocateShapeSpecRank_;
const int rank_;
const int allocateCoarraySpecRank_;
const int corank_;
bool hasDeferredTypeParameter_{false};
bool isUnlimitedPolymorphic_{false};
bool isAbstract_{false};
bool hasAllocateShapeSpecList() const { return allocateShapeSpecRank_ != 0; }
bool hasAllocateCoarraySpec() const { return allocateCoarraySpecRank_ != 0; }
bool RunCoarrayRelatedChecks(SemanticsContext &) const;
@ -88,7 +75,6 @@ private:
}
void GatherAllocationBasicInfo() {
if (type_) {
if (type_->category() == DeclTypeSpec::Category::Character) {
hasDeferredTypeParameter_ =
type_->characterTypeSpec().length().isDeferred();
@ -96,13 +82,23 @@ private:
for (const auto &pair : derivedTypeSpec->parameters()) {
hasDeferredTypeParameter_ |= pair.second.isDeferred();
}
isAbstract_ =
derivedTypeSpec->typeSymbol().attrs().test(Attr::ABSTRACT);
isAbstract_ = derivedTypeSpec->typeSymbol().attrs().test(Attr::ABSTRACT);
}
isUnlimitedPolymorphic_ =
type_->category() == DeclTypeSpec::Category::ClassStar;
}
}
AllocateCheckerInfo &allocateInfo_;
const parser::AllocateObject &allocateObject_;
const parser::Name &name_;
const DeclTypeSpec *type_;
const int allocateShapeSpecRank_;
const int rank_;
const int allocateCoarraySpecRank_;
const int corank_;
bool hasDeferredTypeParameter_{false};
bool isUnlimitedPolymorphic_{false};
bool isAbstract_{false};
};
static std::optional<AllocateCheckerInfo> CheckAllocateOptions(
@ -408,6 +404,7 @@ bool AllocationCheckerHelper::RunChecks(SemanticsContext &context) {
if (type_ == nullptr) {
return false;
}
GatherAllocationBasicInfo();
if (!IsVariableName(*name_.symbol)) { // C932 pre-requisite
context.Say(name_.source,
"Name in ALLOCATE statement must be a variable name"_err_en_US);

9
flang/test/semantics/allocate11.f90
View File

@ -16,7 +16,7 @@
! TODO: Function Pointer in allocate and derived types!
! Rules I should know when working wit coarrays and derived type:
! Rules I should know when working with coarrays and derived type:
! C736: If EXTENDS appears and the type being defined has a coarray ultimate
! component, its parent type shall have a coarray ultimate component.
@ -67,12 +67,13 @@ subroutine C937(var)
class(*), allocatable :: var
! unlimited polymorphic is the ONLY way to get an allocatable/pointer 'var' that can be
! allocated with a type-spec T that has coarray ultimate component.
! Rational:
! allocated with a type-spec T that has coarray ultimate component without
! violating other rules than C937.
! Rationale:
! C934 => var must be type compatible with T.
! => var type is T, a type P extended by T, or unlimited polymorphic
! C825 => var cannot be of type T.
! C736 => all parent type P of T must have a coarray unlimited component
! C736 => all parent types P of T must have a coarray ultimate component
! => var cannot be of type P (C825)
! => if var can be defined, it can only be unlimited polymorphic