1 //===-- lib/Semantics/check-do-forall.cpp ---------------------------------===//
2 //
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4 // See https://llvm.org/LICENSE.txt for license information.
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6 //
7 //===----------------------------------------------------------------------===//
8 
9 #include "check-do-forall.h"
10 #include "flang/Common/template.h"
11 #include "flang/Evaluate/call.h"
12 #include "flang/Evaluate/expression.h"
13 #include "flang/Evaluate/tools.h"
14 #include "flang/Parser/message.h"
15 #include "flang/Parser/parse-tree-visitor.h"
16 #include "flang/Parser/tools.h"
17 #include "flang/Semantics/attr.h"
18 #include "flang/Semantics/scope.h"
19 #include "flang/Semantics/semantics.h"
20 #include "flang/Semantics/symbol.h"
21 #include "flang/Semantics/tools.h"
22 #include "flang/Semantics/type.h"
23 
24 namespace Fortran::evaluate {
25 using ActualArgumentRef = common::Reference<const ActualArgument>;
26 
operator <(ActualArgumentRef x,ActualArgumentRef y)27 inline bool operator<(ActualArgumentRef x, ActualArgumentRef y) {
28   return &*x < &*y;
29 }
30 } // namespace Fortran::evaluate
31 
32 namespace Fortran::semantics {
33 
34 using namespace parser::literals;
35 
36 using Bounds = parser::LoopControl::Bounds;
37 using IndexVarKind = SemanticsContext::IndexVarKind;
38 
GetConcurrentHeader(const parser::LoopControl & loopControl)39 static const parser::ConcurrentHeader &GetConcurrentHeader(
40     const parser::LoopControl &loopControl) {
41   const auto &concurrent{
42       std::get<parser::LoopControl::Concurrent>(loopControl.u)};
43   return std::get<parser::ConcurrentHeader>(concurrent.t);
44 }
GetConcurrentHeader(const parser::ForallConstruct & construct)45 static const parser::ConcurrentHeader &GetConcurrentHeader(
46     const parser::ForallConstruct &construct) {
47   const auto &stmt{
48       std::get<parser::Statement<parser::ForallConstructStmt>>(construct.t)};
49   return std::get<common::Indirection<parser::ConcurrentHeader>>(
50       stmt.statement.t)
51       .value();
52 }
GetConcurrentHeader(const parser::ForallStmt & stmt)53 static const parser::ConcurrentHeader &GetConcurrentHeader(
54     const parser::ForallStmt &stmt) {
55   return std::get<common::Indirection<parser::ConcurrentHeader>>(stmt.t)
56       .value();
57 }
58 template <typename T>
GetControls(const T & x)59 static const std::list<parser::ConcurrentControl> &GetControls(const T &x) {
60   return std::get<std::list<parser::ConcurrentControl>>(
61       GetConcurrentHeader(x).t);
62 }
63 
GetBounds(const parser::DoConstruct & doConstruct)64 static const Bounds &GetBounds(const parser::DoConstruct &doConstruct) {
65   auto &loopControl{doConstruct.GetLoopControl().value()};
66   return std::get<Bounds>(loopControl.u);
67 }
68 
GetDoVariable(const parser::DoConstruct & doConstruct)69 static const parser::Name &GetDoVariable(
70     const parser::DoConstruct &doConstruct) {
71   const Bounds &bounds{GetBounds(doConstruct)};
72   return bounds.name.thing;
73 }
74 
GetEnclosingDoMsg()75 static parser::MessageFixedText GetEnclosingDoMsg() {
76   return "Enclosing DO CONCURRENT statement"_en_US;
77 }
78 
SayWithDo(SemanticsContext & context,parser::CharBlock stmtLocation,parser::MessageFixedText && message,parser::CharBlock doLocation)79 static void SayWithDo(SemanticsContext &context, parser::CharBlock stmtLocation,
80     parser::MessageFixedText &&message, parser::CharBlock doLocation) {
81   context.Say(stmtLocation, message).Attach(doLocation, GetEnclosingDoMsg());
82 }
83 
84 // 11.1.7.5 - enforce semantics constraints on a DO CONCURRENT loop body
85 class DoConcurrentBodyEnforce {
86 public:
DoConcurrentBodyEnforce(SemanticsContext & context,parser::CharBlock doConcurrentSourcePosition)87   DoConcurrentBodyEnforce(
88       SemanticsContext &context, parser::CharBlock doConcurrentSourcePosition)
89       : context_{context}, doConcurrentSourcePosition_{
90                                doConcurrentSourcePosition} {}
labels()91   std::set<parser::Label> labels() { return labels_; }
Pre(const T &)92   template <typename T> bool Pre(const T &) { return true; }
Post(const T &)93   template <typename T> void Post(const T &) {}
94 
Pre(const parser::Statement<T> & statement)95   template <typename T> bool Pre(const parser::Statement<T> &statement) {
96     currentStatementSourcePosition_ = statement.source;
97     if (statement.label.has_value()) {
98       labels_.insert(*statement.label);
99     }
100     return true;
101   }
102 
Pre(const parser::UnlabeledStatement<T> & stmt)103   template <typename T> bool Pre(const parser::UnlabeledStatement<T> &stmt) {
104     currentStatementSourcePosition_ = stmt.source;
105     return true;
106   }
107 
108   // C1140 -- Can't deallocate a polymorphic entity in a DO CONCURRENT.
109   // Deallocation can be caused by exiting a block that declares an allocatable
110   // entity, assignment to an allocatable variable, or an actual DEALLOCATE
111   // statement
112   //
113   // Note also that the deallocation of a derived type entity might cause the
114   // invocation of an IMPURE final subroutine. (C1139)
115   //
116 
117   // Only to be called for symbols with ObjectEntityDetails
HasImpureFinal(const Symbol & original)118   static bool HasImpureFinal(const Symbol &original) {
119     const Symbol &symbol{ResolveAssociations(original)};
120     if (symbol.has<ObjectEntityDetails>()) {
121       if (const DeclTypeSpec * symType{symbol.GetType()}) {
122         if (const DerivedTypeSpec * derived{symType->AsDerived()}) {
123           return semantics::HasImpureFinal(*derived);
124         }
125       }
126     }
127     return false;
128   }
129 
130   // Predicate for deallocations caused by block exit and direct deallocation
DeallocateAll(const Symbol &)131   static bool DeallocateAll(const Symbol &) { return true; }
132 
133   // Predicate for deallocations caused by intrinsic assignment
DeallocateNonCoarray(const Symbol & component)134   static bool DeallocateNonCoarray(const Symbol &component) {
135     return !IsCoarray(component);
136   }
137 
WillDeallocatePolymorphic(const Symbol & entity,const std::function<bool (const Symbol &)> & WillDeallocate)138   static bool WillDeallocatePolymorphic(const Symbol &entity,
139       const std::function<bool(const Symbol &)> &WillDeallocate) {
140     return WillDeallocate(entity) && IsPolymorphicAllocatable(entity);
141   }
142 
143   // Is it possible that we will we deallocate a polymorphic entity or one
144   // of its components?
MightDeallocatePolymorphic(const Symbol & original,const std::function<bool (const Symbol &)> & WillDeallocate)145   static bool MightDeallocatePolymorphic(const Symbol &original,
146       const std::function<bool(const Symbol &)> &WillDeallocate) {
147     const Symbol &symbol{ResolveAssociations(original)};
148     // Check the entity itself, no coarray exception here
149     if (IsPolymorphicAllocatable(symbol)) {
150       return true;
151     }
152     // Check the components
153     if (const auto *details{symbol.detailsIf<ObjectEntityDetails>()}) {
154       if (const DeclTypeSpec * entityType{details->type()}) {
155         if (const DerivedTypeSpec * derivedType{entityType->AsDerived()}) {
156           UltimateComponentIterator ultimates{*derivedType};
157           for (const auto &ultimate : ultimates) {
158             if (WillDeallocatePolymorphic(ultimate, WillDeallocate)) {
159               return true;
160             }
161           }
162         }
163       }
164     }
165     return false;
166   }
167 
SayDeallocateWithImpureFinal(const Symbol & entity,const char * reason)168   void SayDeallocateWithImpureFinal(const Symbol &entity, const char *reason) {
169     context_.SayWithDecl(entity, currentStatementSourcePosition_,
170         "Deallocation of an entity with an IMPURE FINAL procedure"
171         " caused by %s not allowed in DO CONCURRENT"_err_en_US,
172         reason);
173   }
174 
SayDeallocateOfPolymorph(parser::CharBlock location,const Symbol & entity,const char * reason)175   void SayDeallocateOfPolymorph(
176       parser::CharBlock location, const Symbol &entity, const char *reason) {
177     context_.SayWithDecl(entity, location,
178         "Deallocation of a polymorphic entity caused by %s"
179         " not allowed in DO CONCURRENT"_err_en_US,
180         reason);
181   }
182 
183   // Deallocation caused by block exit
184   // Allocatable entities and all of their allocatable subcomponents will be
185   // deallocated.  This test is different from the other two because it does
186   // not deallocate in cases where the entity itself is not allocatable but
187   // has allocatable polymorphic components
Post(const parser::BlockConstruct & blockConstruct)188   void Post(const parser::BlockConstruct &blockConstruct) {
189     const auto &endBlockStmt{
190         std::get<parser::Statement<parser::EndBlockStmt>>(blockConstruct.t)};
191     const Scope &blockScope{context_.FindScope(endBlockStmt.source)};
192     const Scope &doScope{context_.FindScope(doConcurrentSourcePosition_)};
193     if (DoesScopeContain(&doScope, blockScope)) {
194       const char *reason{"block exit"};
195       for (auto &pair : blockScope) {
196         const Symbol &entity{*pair.second};
197         if (IsAllocatable(entity) && !IsSaved(entity) &&
198             MightDeallocatePolymorphic(entity, DeallocateAll)) {
199           SayDeallocateOfPolymorph(endBlockStmt.source, entity, reason);
200         }
201         if (HasImpureFinal(entity)) {
202           SayDeallocateWithImpureFinal(entity, reason);
203         }
204       }
205     }
206   }
207 
208   // Deallocation caused by assignment
209   // Note that this case does not cause deallocation of coarray components
Post(const parser::AssignmentStmt & stmt)210   void Post(const parser::AssignmentStmt &stmt) {
211     const auto &variable{std::get<parser::Variable>(stmt.t)};
212     if (const Symbol * entity{GetLastName(variable).symbol}) {
213       const char *reason{"assignment"};
214       if (MightDeallocatePolymorphic(*entity, DeallocateNonCoarray)) {
215         SayDeallocateOfPolymorph(variable.GetSource(), *entity, reason);
216       }
217       if (HasImpureFinal(*entity)) {
218         SayDeallocateWithImpureFinal(*entity, reason);
219       }
220     }
221   }
222 
223   // Deallocation from a DEALLOCATE statement
224   // This case is different because DEALLOCATE statements deallocate both
225   // ALLOCATABLE and POINTER entities
Post(const parser::DeallocateStmt & stmt)226   void Post(const parser::DeallocateStmt &stmt) {
227     const auto &allocateObjectList{
228         std::get<std::list<parser::AllocateObject>>(stmt.t)};
229     for (const auto &allocateObject : allocateObjectList) {
230       const parser::Name &name{GetLastName(allocateObject)};
231       const char *reason{"a DEALLOCATE statement"};
232       if (name.symbol) {
233         const Symbol &entity{*name.symbol};
234         const DeclTypeSpec *entityType{entity.GetType()};
235         if ((entityType && entityType->IsPolymorphic()) || // POINTER case
236             MightDeallocatePolymorphic(entity, DeallocateAll)) {
237           SayDeallocateOfPolymorph(
238               currentStatementSourcePosition_, entity, reason);
239         }
240         if (HasImpureFinal(entity)) {
241           SayDeallocateWithImpureFinal(entity, reason);
242         }
243       }
244     }
245   }
246 
247   // C1137 -- No image control statements in a DO CONCURRENT
Post(const parser::ExecutableConstruct & construct)248   void Post(const parser::ExecutableConstruct &construct) {
249     if (IsImageControlStmt(construct)) {
250       const parser::CharBlock statementLocation{
251           GetImageControlStmtLocation(construct)};
252       auto &msg{context_.Say(statementLocation,
253           "An image control statement is not allowed in DO"
254           " CONCURRENT"_err_en_US)};
255       if (auto coarrayMsg{GetImageControlStmtCoarrayMsg(construct)}) {
256         msg.Attach(statementLocation, *coarrayMsg);
257       }
258       msg.Attach(doConcurrentSourcePosition_, GetEnclosingDoMsg());
259     }
260   }
261 
262   // C1136 -- No RETURN statements in a DO CONCURRENT
Post(const parser::ReturnStmt &)263   void Post(const parser::ReturnStmt &) {
264     context_
265         .Say(currentStatementSourcePosition_,
266             "RETURN is not allowed in DO CONCURRENT"_err_en_US)
267         .Attach(doConcurrentSourcePosition_, GetEnclosingDoMsg());
268   }
269 
270   // C1139: call to impure procedure and ...
271   // C1141: cannot call ieee_get_flag, ieee_[gs]et_halting_mode
272   // It's not necessary to check the ieee_get* procedures because they're
273   // not pure, and impure procedures are caught by checks for constraint C1139
Post(const parser::ProcedureDesignator & procedureDesignator)274   void Post(const parser::ProcedureDesignator &procedureDesignator) {
275     if (auto *name{std::get_if<parser::Name>(&procedureDesignator.u)}) {
276       if (name->symbol && !IsPureProcedure(*name->symbol)) {
277         SayWithDo(context_, currentStatementSourcePosition_,
278             "Call to an impure procedure is not allowed in DO"
279             " CONCURRENT"_err_en_US,
280             doConcurrentSourcePosition_);
281       }
282       if (name->symbol && fromScope(*name->symbol, "ieee_exceptions"s)) {
283         if (name->source == "ieee_set_halting_mode") {
284           SayWithDo(context_, currentStatementSourcePosition_,
285               "IEEE_SET_HALTING_MODE is not allowed in DO "
286               "CONCURRENT"_err_en_US,
287               doConcurrentSourcePosition_);
288         }
289       }
290     } else {
291       // C1139: this a procedure component
292       auto &component{std::get<parser::ProcComponentRef>(procedureDesignator.u)
293                           .v.thing.component};
294       if (component.symbol && !IsPureProcedure(*component.symbol)) {
295         SayWithDo(context_, currentStatementSourcePosition_,
296             "Call to an impure procedure component is not allowed"
297             " in DO CONCURRENT"_err_en_US,
298             doConcurrentSourcePosition_);
299       }
300     }
301   }
302 
303   // 11.1.7.5, paragraph 5, no ADVANCE specifier in a DO CONCURRENT
Post(const parser::IoControlSpec & ioControlSpec)304   void Post(const parser::IoControlSpec &ioControlSpec) {
305     if (auto *charExpr{
306             std::get_if<parser::IoControlSpec::CharExpr>(&ioControlSpec.u)}) {
307       if (std::get<parser::IoControlSpec::CharExpr::Kind>(charExpr->t) ==
308           parser::IoControlSpec::CharExpr::Kind::Advance) {
309         SayWithDo(context_, currentStatementSourcePosition_,
310             "ADVANCE specifier is not allowed in DO"
311             " CONCURRENT"_err_en_US,
312             doConcurrentSourcePosition_);
313       }
314     }
315   }
316 
317 private:
fromScope(const Symbol & symbol,const std::string & moduleName)318   bool fromScope(const Symbol &symbol, const std::string &moduleName) {
319     if (symbol.GetUltimate().owner().IsModule() &&
320         symbol.GetUltimate().owner().GetName().value().ToString() ==
321             moduleName) {
322       return true;
323     }
324     return false;
325   }
326 
327   std::set<parser::Label> labels_;
328   parser::CharBlock currentStatementSourcePosition_;
329   SemanticsContext &context_;
330   parser::CharBlock doConcurrentSourcePosition_;
331 }; // class DoConcurrentBodyEnforce
332 
333 // Class for enforcing C1130 -- in a DO CONCURRENT with DEFAULT(NONE),
334 // variables from enclosing scopes must have their locality specified
335 class DoConcurrentVariableEnforce {
336 public:
DoConcurrentVariableEnforce(SemanticsContext & context,parser::CharBlock doConcurrentSourcePosition)337   DoConcurrentVariableEnforce(
338       SemanticsContext &context, parser::CharBlock doConcurrentSourcePosition)
339       : context_{context},
340         doConcurrentSourcePosition_{doConcurrentSourcePosition},
341         blockScope_{context.FindScope(doConcurrentSourcePosition_)} {}
342 
Pre(const T &)343   template <typename T> bool Pre(const T &) { return true; }
Post(const T &)344   template <typename T> void Post(const T &) {}
345 
346   // Check to see if the name is a variable from an enclosing scope
Post(const parser::Name & name)347   void Post(const parser::Name &name) {
348     if (const Symbol * symbol{name.symbol}) {
349       if (IsVariableName(*symbol)) {
350         const Scope &variableScope{symbol->owner()};
351         if (DoesScopeContain(&variableScope, blockScope_)) {
352           context_.SayWithDecl(*symbol, name.source,
353               "Variable '%s' from an enclosing scope referenced in DO "
354               "CONCURRENT with DEFAULT(NONE) must appear in a "
355               "locality-spec"_err_en_US,
356               symbol->name());
357         }
358       }
359     }
360   }
361 
362 private:
363   SemanticsContext &context_;
364   parser::CharBlock doConcurrentSourcePosition_;
365   const Scope &blockScope_;
366 }; // class DoConcurrentVariableEnforce
367 
368 // Find a DO or FORALL and enforce semantics checks on its body
369 class DoContext {
370 public:
DoContext(SemanticsContext & context,IndexVarKind kind)371   DoContext(SemanticsContext &context, IndexVarKind kind)
372       : context_{context}, kind_{kind} {}
373 
374   // Mark this DO construct as a point of definition for the DO variables
375   // or index-names it contains.  If they're already defined, emit an error
376   // message.  We need to remember both the variable and the source location of
377   // the variable in the DO construct so that we can remove it when we leave
378   // the DO construct and use its location in error messages.
DefineDoVariables(const parser::DoConstruct & doConstruct)379   void DefineDoVariables(const parser::DoConstruct &doConstruct) {
380     if (doConstruct.IsDoNormal()) {
381       context_.ActivateIndexVar(GetDoVariable(doConstruct), IndexVarKind::DO);
382     } else if (doConstruct.IsDoConcurrent()) {
383       if (const auto &loopControl{doConstruct.GetLoopControl()}) {
384         ActivateIndexVars(GetControls(*loopControl));
385       }
386     }
387   }
388 
389   // Called at the end of a DO construct to deactivate the DO construct
ResetDoVariables(const parser::DoConstruct & doConstruct)390   void ResetDoVariables(const parser::DoConstruct &doConstruct) {
391     if (doConstruct.IsDoNormal()) {
392       context_.DeactivateIndexVar(GetDoVariable(doConstruct));
393     } else if (doConstruct.IsDoConcurrent()) {
394       if (const auto &loopControl{doConstruct.GetLoopControl()}) {
395         DeactivateIndexVars(GetControls(*loopControl));
396       }
397     }
398   }
399 
ActivateIndexVars(const std::list<parser::ConcurrentControl> & controls)400   void ActivateIndexVars(const std::list<parser::ConcurrentControl> &controls) {
401     for (const auto &control : controls) {
402       context_.ActivateIndexVar(std::get<parser::Name>(control.t), kind_);
403     }
404   }
DeactivateIndexVars(const std::list<parser::ConcurrentControl> & controls)405   void DeactivateIndexVars(
406       const std::list<parser::ConcurrentControl> &controls) {
407     for (const auto &control : controls) {
408       context_.DeactivateIndexVar(std::get<parser::Name>(control.t));
409     }
410   }
411 
Check(const parser::DoConstruct & doConstruct)412   void Check(const parser::DoConstruct &doConstruct) {
413     if (doConstruct.IsDoConcurrent()) {
414       CheckDoConcurrent(doConstruct);
415       return;
416     }
417     if (doConstruct.IsDoNormal()) {
418       CheckDoNormal(doConstruct);
419       return;
420     }
421     // TODO: handle the other cases
422   }
423 
Check(const parser::ForallStmt & stmt)424   void Check(const parser::ForallStmt &stmt) {
425     CheckConcurrentHeader(GetConcurrentHeader(stmt));
426   }
Check(const parser::ForallConstruct & construct)427   void Check(const parser::ForallConstruct &construct) {
428     CheckConcurrentHeader(GetConcurrentHeader(construct));
429   }
430 
Check(const parser::ForallAssignmentStmt & stmt)431   void Check(const parser::ForallAssignmentStmt &stmt) {
432     const evaluate::Assignment *assignment{std::visit(
433         common::visitors{[&](const auto &x) { return GetAssignment(x); }},
434         stmt.u)};
435     if (assignment) {
436       CheckForallIndexesUsed(*assignment);
437       CheckForImpureCall(assignment->lhs);
438       CheckForImpureCall(assignment->rhs);
439       if (const auto *proc{
440               std::get_if<evaluate::ProcedureRef>(&assignment->u)}) {
441         CheckForImpureCall(*proc);
442       }
443       std::visit(common::visitors{
444                      [](const evaluate::Assignment::Intrinsic &) {},
445                      [&](const evaluate::ProcedureRef &proc) {
446                        CheckForImpureCall(proc);
447                      },
448                      [&](const evaluate::Assignment::BoundsSpec &bounds) {
449                        for (const auto &bound : bounds) {
450                          CheckForImpureCall(SomeExpr{bound});
451                        }
452                      },
453                      [&](const evaluate::Assignment::BoundsRemapping &bounds) {
454                        for (const auto &bound : bounds) {
455                          CheckForImpureCall(SomeExpr{bound.first});
456                          CheckForImpureCall(SomeExpr{bound.second});
457                        }
458                      },
459                  },
460           assignment->u);
461     }
462   }
463 
464 private:
SayBadDoControl(parser::CharBlock sourceLocation)465   void SayBadDoControl(parser::CharBlock sourceLocation) {
466     context_.Say(sourceLocation, "DO controls should be INTEGER"_err_en_US);
467   }
468 
CheckDoControl(const parser::CharBlock & sourceLocation,bool isReal)469   void CheckDoControl(const parser::CharBlock &sourceLocation, bool isReal) {
470     const bool warn{context_.warnOnNonstandardUsage() ||
471         context_.ShouldWarn(common::LanguageFeature::RealDoControls)};
472     if (isReal && !warn) {
473       // No messages for the default case
474     } else if (isReal && warn) {
475       context_.Say(sourceLocation, "DO controls should be INTEGER"_en_US);
476     } else {
477       SayBadDoControl(sourceLocation);
478     }
479   }
480 
CheckDoVariable(const parser::ScalarName & scalarName)481   void CheckDoVariable(const parser::ScalarName &scalarName) {
482     const parser::CharBlock &sourceLocation{scalarName.thing.source};
483     if (const Symbol * symbol{scalarName.thing.symbol}) {
484       if (!IsVariableName(*symbol)) {
485         context_.Say(
486             sourceLocation, "DO control must be an INTEGER variable"_err_en_US);
487       } else {
488         const DeclTypeSpec *symType{symbol->GetType()};
489         if (!symType) {
490           SayBadDoControl(sourceLocation);
491         } else {
492           if (!symType->IsNumeric(TypeCategory::Integer)) {
493             CheckDoControl(
494                 sourceLocation, symType->IsNumeric(TypeCategory::Real));
495           }
496         }
497       } // No messages for INTEGER
498     }
499   }
500 
501   // Semantic checks for the limit and step expressions
CheckDoExpression(const parser::ScalarExpr & scalarExpression)502   void CheckDoExpression(const parser::ScalarExpr &scalarExpression) {
503     if (const SomeExpr * expr{GetExpr(scalarExpression)}) {
504       if (!ExprHasTypeCategory(*expr, TypeCategory::Integer)) {
505         // No warnings or errors for type INTEGER
506         const parser::CharBlock &loc{scalarExpression.thing.value().source};
507         CheckDoControl(loc, ExprHasTypeCategory(*expr, TypeCategory::Real));
508       }
509     }
510   }
511 
CheckDoNormal(const parser::DoConstruct & doConstruct)512   void CheckDoNormal(const parser::DoConstruct &doConstruct) {
513     // C1120 -- types of DO variables must be INTEGER, extended by allowing
514     // REAL and DOUBLE PRECISION
515     const Bounds &bounds{GetBounds(doConstruct)};
516     CheckDoVariable(bounds.name);
517     CheckDoExpression(bounds.lower);
518     CheckDoExpression(bounds.upper);
519     if (bounds.step) {
520       CheckDoExpression(*bounds.step);
521       if (IsZero(*bounds.step)) {
522         context_.Say(bounds.step->thing.value().source,
523             "DO step expression should not be zero"_en_US);
524       }
525     }
526   }
527 
CheckDoConcurrent(const parser::DoConstruct & doConstruct)528   void CheckDoConcurrent(const parser::DoConstruct &doConstruct) {
529     auto &doStmt{
530         std::get<parser::Statement<parser::NonLabelDoStmt>>(doConstruct.t)};
531     currentStatementSourcePosition_ = doStmt.source;
532 
533     const parser::Block &block{std::get<parser::Block>(doConstruct.t)};
534     DoConcurrentBodyEnforce doConcurrentBodyEnforce{context_, doStmt.source};
535     parser::Walk(block, doConcurrentBodyEnforce);
536 
537     LabelEnforce doConcurrentLabelEnforce{context_,
538         doConcurrentBodyEnforce.labels(), currentStatementSourcePosition_,
539         "DO CONCURRENT"};
540     parser::Walk(block, doConcurrentLabelEnforce);
541 
542     const auto &loopControl{doConstruct.GetLoopControl()};
543     CheckConcurrentLoopControl(*loopControl);
544     CheckLocalitySpecs(*loopControl, block);
545   }
546 
547   // Return a set of symbols whose names are in a Local locality-spec.  Look
548   // the names up in the scope that encloses the DO construct to avoid getting
549   // the local versions of them.  Then follow the host-, use-, and
550   // construct-associations to get the root symbols
GatherLocals(const std::list<parser::LocalitySpec> & localitySpecs) const551   UnorderedSymbolSet GatherLocals(
552       const std::list<parser::LocalitySpec> &localitySpecs) const {
553     UnorderedSymbolSet symbols;
554     const Scope &parentScope{
555         context_.FindScope(currentStatementSourcePosition_).parent()};
556     // Loop through the LocalitySpec::Local locality-specs
557     for (const auto &ls : localitySpecs) {
558       if (const auto *names{std::get_if<parser::LocalitySpec::Local>(&ls.u)}) {
559         // Loop through the names in the Local locality-spec getting their
560         // symbols
561         for (const parser::Name &name : names->v) {
562           if (const Symbol * symbol{parentScope.FindSymbol(name.source)}) {
563             symbols.insert(ResolveAssociations(*symbol));
564           }
565         }
566       }
567     }
568     return symbols;
569   }
570 
GatherSymbolsFromExpression(const parser::Expr & expression)571   static UnorderedSymbolSet GatherSymbolsFromExpression(
572       const parser::Expr &expression) {
573     UnorderedSymbolSet result;
574     if (const auto *expr{GetExpr(expression)}) {
575       for (const Symbol &symbol : evaluate::CollectSymbols(*expr)) {
576         result.insert(ResolveAssociations(symbol));
577       }
578     }
579     return result;
580   }
581 
582   // C1121 - procedures in mask must be pure
CheckMaskIsPure(const parser::ScalarLogicalExpr & mask) const583   void CheckMaskIsPure(const parser::ScalarLogicalExpr &mask) const {
584     UnorderedSymbolSet references{
585         GatherSymbolsFromExpression(mask.thing.thing.value())};
586     for (const Symbol &ref : OrderBySourcePosition(references)) {
587       if (IsProcedure(ref) && !IsPureProcedure(ref)) {
588         context_.SayWithDecl(ref, parser::Unwrap<parser::Expr>(mask)->source,
589             "%s mask expression may not reference impure procedure '%s'"_err_en_US,
590             LoopKindName(), ref.name());
591         return;
592       }
593     }
594   }
595 
CheckNoCollisions(const UnorderedSymbolSet & refs,const UnorderedSymbolSet & uses,parser::MessageFixedText && errorMessage,const parser::CharBlock & refPosition) const596   void CheckNoCollisions(const UnorderedSymbolSet &refs,
597       const UnorderedSymbolSet &uses, parser::MessageFixedText &&errorMessage,
598       const parser::CharBlock &refPosition) const {
599     for (const Symbol &ref : OrderBySourcePosition(refs)) {
600       if (uses.find(ref) != uses.end()) {
601         context_.SayWithDecl(ref, refPosition, std::move(errorMessage),
602             LoopKindName(), ref.name());
603         return;
604       }
605     }
606   }
607 
HasNoReferences(const UnorderedSymbolSet & indexNames,const parser::ScalarIntExpr & expr) const608   void HasNoReferences(const UnorderedSymbolSet &indexNames,
609       const parser::ScalarIntExpr &expr) const {
610     CheckNoCollisions(GatherSymbolsFromExpression(expr.thing.thing.value()),
611         indexNames,
612         "%s limit expression may not reference index variable '%s'"_err_en_US,
613         expr.thing.thing.value().source);
614   }
615 
616   // C1129, names in local locality-specs can't be in mask expressions
CheckMaskDoesNotReferenceLocal(const parser::ScalarLogicalExpr & mask,const UnorderedSymbolSet & localVars) const617   void CheckMaskDoesNotReferenceLocal(const parser::ScalarLogicalExpr &mask,
618       const UnorderedSymbolSet &localVars) const {
619     CheckNoCollisions(GatherSymbolsFromExpression(mask.thing.thing.value()),
620         localVars,
621         "%s mask expression references variable '%s'"
622         " in LOCAL locality-spec"_err_en_US,
623         mask.thing.thing.value().source);
624   }
625 
626   // C1129, names in local locality-specs can't be in limit or step
627   // expressions
CheckExprDoesNotReferenceLocal(const parser::ScalarIntExpr & expr,const UnorderedSymbolSet & localVars) const628   void CheckExprDoesNotReferenceLocal(const parser::ScalarIntExpr &expr,
629       const UnorderedSymbolSet &localVars) const {
630     CheckNoCollisions(GatherSymbolsFromExpression(expr.thing.thing.value()),
631         localVars,
632         "%s expression references variable '%s'"
633         " in LOCAL locality-spec"_err_en_US,
634         expr.thing.thing.value().source);
635   }
636 
637   // C1130, DEFAULT(NONE) locality requires names to be in locality-specs to
638   // be used in the body of the DO loop
CheckDefaultNoneImpliesExplicitLocality(const std::list<parser::LocalitySpec> & localitySpecs,const parser::Block & block) const639   void CheckDefaultNoneImpliesExplicitLocality(
640       const std::list<parser::LocalitySpec> &localitySpecs,
641       const parser::Block &block) const {
642     bool hasDefaultNone{false};
643     for (auto &ls : localitySpecs) {
644       if (std::holds_alternative<parser::LocalitySpec::DefaultNone>(ls.u)) {
645         if (hasDefaultNone) {
646           // C1127, you can only have one DEFAULT(NONE)
647           context_.Say(currentStatementSourcePosition_,
648               "Only one DEFAULT(NONE) may appear"_en_US);
649           break;
650         }
651         hasDefaultNone = true;
652       }
653     }
654     if (hasDefaultNone) {
655       DoConcurrentVariableEnforce doConcurrentVariableEnforce{
656           context_, currentStatementSourcePosition_};
657       parser::Walk(block, doConcurrentVariableEnforce);
658     }
659   }
660 
661   // C1123, concurrent limit or step expressions can't reference index-names
CheckConcurrentHeader(const parser::ConcurrentHeader & header) const662   void CheckConcurrentHeader(const parser::ConcurrentHeader &header) const {
663     if (const auto &mask{
664             std::get<std::optional<parser::ScalarLogicalExpr>>(header.t)}) {
665       CheckMaskIsPure(*mask);
666     }
667     auto &controls{std::get<std::list<parser::ConcurrentControl>>(header.t)};
668     UnorderedSymbolSet indexNames;
669     for (const parser::ConcurrentControl &control : controls) {
670       const auto &indexName{std::get<parser::Name>(control.t)};
671       if (indexName.symbol) {
672         indexNames.insert(*indexName.symbol);
673       }
674     }
675     if (!indexNames.empty()) {
676       for (const parser::ConcurrentControl &control : controls) {
677         HasNoReferences(indexNames, std::get<1>(control.t));
678         HasNoReferences(indexNames, std::get<2>(control.t));
679         if (const auto &intExpr{
680                 std::get<std::optional<parser::ScalarIntExpr>>(control.t)}) {
681           const parser::Expr &expr{intExpr->thing.thing.value()};
682           CheckNoCollisions(GatherSymbolsFromExpression(expr), indexNames,
683               "%s step expression may not reference index variable '%s'"_err_en_US,
684               expr.source);
685           if (IsZero(expr)) {
686             context_.Say(expr.source,
687                 "%s step expression may not be zero"_err_en_US, LoopKindName());
688           }
689         }
690       }
691     }
692   }
693 
CheckLocalitySpecs(const parser::LoopControl & control,const parser::Block & block) const694   void CheckLocalitySpecs(
695       const parser::LoopControl &control, const parser::Block &block) const {
696     const auto &concurrent{
697         std::get<parser::LoopControl::Concurrent>(control.u)};
698     const auto &header{std::get<parser::ConcurrentHeader>(concurrent.t)};
699     const auto &localitySpecs{
700         std::get<std::list<parser::LocalitySpec>>(concurrent.t)};
701     if (!localitySpecs.empty()) {
702       const UnorderedSymbolSet &localVars{GatherLocals(localitySpecs)};
703       for (const auto &c : GetControls(control)) {
704         CheckExprDoesNotReferenceLocal(std::get<1>(c.t), localVars);
705         CheckExprDoesNotReferenceLocal(std::get<2>(c.t), localVars);
706         if (const auto &expr{
707                 std::get<std::optional<parser::ScalarIntExpr>>(c.t)}) {
708           CheckExprDoesNotReferenceLocal(*expr, localVars);
709         }
710       }
711       if (const auto &mask{
712               std::get<std::optional<parser::ScalarLogicalExpr>>(header.t)}) {
713         CheckMaskDoesNotReferenceLocal(*mask, localVars);
714       }
715       CheckDefaultNoneImpliesExplicitLocality(localitySpecs, block);
716     }
717   }
718 
719   // check constraints [C1121 .. C1130]
CheckConcurrentLoopControl(const parser::LoopControl & control) const720   void CheckConcurrentLoopControl(const parser::LoopControl &control) const {
721     const auto &concurrent{
722         std::get<parser::LoopControl::Concurrent>(control.u)};
723     CheckConcurrentHeader(std::get<parser::ConcurrentHeader>(concurrent.t));
724   }
725 
CheckForImpureCall(const T & x)726   template <typename T> void CheckForImpureCall(const T &x) {
727     if (auto bad{FindImpureCall(context_.foldingContext(), x)}) {
728       context_.Say(
729           "Impure procedure '%s' may not be referenced in a %s"_err_en_US, *bad,
730           LoopKindName());
731     }
732   }
733 
734   // Each index should be used on the LHS of each assignment in a FORALL
CheckForallIndexesUsed(const evaluate::Assignment & assignment)735   void CheckForallIndexesUsed(const evaluate::Assignment &assignment) {
736     SymbolVector indexVars{context_.GetIndexVars(IndexVarKind::FORALL)};
737     if (!indexVars.empty()) {
738       UnorderedSymbolSet symbols{evaluate::CollectSymbols(assignment.lhs)};
739       std::visit(
740           common::visitors{
741               [&](const evaluate::Assignment::BoundsSpec &spec) {
742                 for (const auto &bound : spec) {
743 // TODO: this is working around missing std::set::merge in some versions of
744 // clang that we are building with
745 #ifdef __clang__
746                   auto boundSymbols{evaluate::CollectSymbols(bound)};
747                   symbols.insert(boundSymbols.begin(), boundSymbols.end());
748 #else
749                   symbols.merge(evaluate::CollectSymbols(bound));
750 #endif
751                 }
752               },
753               [&](const evaluate::Assignment::BoundsRemapping &remapping) {
754                 for (const auto &bounds : remapping) {
755 #ifdef __clang__
756                   auto lbSymbols{evaluate::CollectSymbols(bounds.first)};
757                   symbols.insert(lbSymbols.begin(), lbSymbols.end());
758                   auto ubSymbols{evaluate::CollectSymbols(bounds.second)};
759                   symbols.insert(ubSymbols.begin(), ubSymbols.end());
760 #else
761                   symbols.merge(evaluate::CollectSymbols(bounds.first));
762                   symbols.merge(evaluate::CollectSymbols(bounds.second));
763 #endif
764                 }
765               },
766               [](const auto &) {},
767           },
768           assignment.u);
769       for (const Symbol &index : indexVars) {
770         if (symbols.count(index) == 0) {
771           context_.Say(
772               "Warning: FORALL index variable '%s' not used on left-hand side"
773               " of assignment"_en_US,
774               index.name());
775         }
776       }
777     }
778   }
779 
780   // For messages where the DO loop must be DO CONCURRENT, make that explicit.
LoopKindName() const781   const char *LoopKindName() const {
782     return kind_ == IndexVarKind::DO ? "DO CONCURRENT" : "FORALL";
783   }
784 
785   SemanticsContext &context_;
786   const IndexVarKind kind_;
787   parser::CharBlock currentStatementSourcePosition_;
788 }; // class DoContext
789 
Enter(const parser::DoConstruct & doConstruct)790 void DoForallChecker::Enter(const parser::DoConstruct &doConstruct) {
791   DoContext doContext{context_, IndexVarKind::DO};
792   doContext.DefineDoVariables(doConstruct);
793 }
794 
Leave(const parser::DoConstruct & doConstruct)795 void DoForallChecker::Leave(const parser::DoConstruct &doConstruct) {
796   DoContext doContext{context_, IndexVarKind::DO};
797   doContext.Check(doConstruct);
798   doContext.ResetDoVariables(doConstruct);
799 }
800 
Enter(const parser::ForallConstruct & construct)801 void DoForallChecker::Enter(const parser::ForallConstruct &construct) {
802   DoContext doContext{context_, IndexVarKind::FORALL};
803   doContext.ActivateIndexVars(GetControls(construct));
804 }
Leave(const parser::ForallConstruct & construct)805 void DoForallChecker::Leave(const parser::ForallConstruct &construct) {
806   DoContext doContext{context_, IndexVarKind::FORALL};
807   doContext.Check(construct);
808   doContext.DeactivateIndexVars(GetControls(construct));
809 }
810 
Enter(const parser::ForallStmt & stmt)811 void DoForallChecker::Enter(const parser::ForallStmt &stmt) {
812   DoContext doContext{context_, IndexVarKind::FORALL};
813   doContext.ActivateIndexVars(GetControls(stmt));
814 }
Leave(const parser::ForallStmt & stmt)815 void DoForallChecker::Leave(const parser::ForallStmt &stmt) {
816   DoContext doContext{context_, IndexVarKind::FORALL};
817   doContext.Check(stmt);
818   doContext.DeactivateIndexVars(GetControls(stmt));
819 }
Leave(const parser::ForallAssignmentStmt & stmt)820 void DoForallChecker::Leave(const parser::ForallAssignmentStmt &stmt) {
821   DoContext doContext{context_, IndexVarKind::FORALL};
822   doContext.Check(stmt);
823 }
824 
825 template <typename A>
GetConstructPosition(const A & a)826 static parser::CharBlock GetConstructPosition(const A &a) {
827   return std::get<0>(a.t).source;
828 }
829 
GetNodePosition(const ConstructNode & construct)830 static parser::CharBlock GetNodePosition(const ConstructNode &construct) {
831   return std::visit(
832       [&](const auto &x) { return GetConstructPosition(*x); }, construct);
833 }
834 
SayBadLeave(StmtType stmtType,const char * enclosingStmtName,const ConstructNode & construct) const835 void DoForallChecker::SayBadLeave(StmtType stmtType,
836     const char *enclosingStmtName, const ConstructNode &construct) const {
837   context_
838       .Say("%s must not leave a %s statement"_err_en_US, EnumToString(stmtType),
839           enclosingStmtName)
840       .Attach(GetNodePosition(construct), "The construct that was left"_en_US);
841 }
842 
MaybeGetDoConstruct(const ConstructNode & construct)843 static const parser::DoConstruct *MaybeGetDoConstruct(
844     const ConstructNode &construct) {
845   if (const auto *doNode{
846           std::get_if<const parser::DoConstruct *>(&construct)}) {
847     return *doNode;
848   } else {
849     return nullptr;
850   }
851 }
852 
ConstructIsDoConcurrent(const ConstructNode & construct)853 static bool ConstructIsDoConcurrent(const ConstructNode &construct) {
854   const parser::DoConstruct *doConstruct{MaybeGetDoConstruct(construct)};
855   return doConstruct && doConstruct->IsDoConcurrent();
856 }
857 
858 // Check that CYCLE and EXIT statements do not cause flow of control to
859 // leave DO CONCURRENT, CRITICAL, or CHANGE TEAM constructs.
CheckForBadLeave(StmtType stmtType,const ConstructNode & construct) const860 void DoForallChecker::CheckForBadLeave(
861     StmtType stmtType, const ConstructNode &construct) const {
862   std::visit(common::visitors{
863                  [&](const parser::DoConstruct *doConstructPtr) {
864                    if (doConstructPtr->IsDoConcurrent()) {
865                      // C1135 and C1167 -- CYCLE and EXIT statements can't leave
866                      // a DO CONCURRENT
867                      SayBadLeave(stmtType, "DO CONCURRENT", construct);
868                    }
869                  },
870                  [&](const parser::CriticalConstruct *) {
871                    // C1135 and C1168 -- similarly, for CRITICAL
872                    SayBadLeave(stmtType, "CRITICAL", construct);
873                  },
874                  [&](const parser::ChangeTeamConstruct *) {
875                    // C1135 and C1168 -- similarly, for CHANGE TEAM
876                    SayBadLeave(stmtType, "CHANGE TEAM", construct);
877                  },
878                  [](const auto *) {},
879              },
880       construct);
881 }
882 
StmtMatchesConstruct(const parser::Name * stmtName,StmtType stmtType,const std::optional<parser::Name> & constructName,const ConstructNode & construct)883 static bool StmtMatchesConstruct(const parser::Name *stmtName,
884     StmtType stmtType, const std::optional<parser::Name> &constructName,
885     const ConstructNode &construct) {
886   bool inDoConstruct{MaybeGetDoConstruct(construct) != nullptr};
887   if (!stmtName) {
888     return inDoConstruct; // Unlabeled statements match all DO constructs
889   } else if (constructName && constructName->source == stmtName->source) {
890     return stmtType == StmtType::EXIT || inDoConstruct;
891   } else {
892     return false;
893   }
894 }
895 
896 // C1167 Can't EXIT from a DO CONCURRENT
CheckDoConcurrentExit(StmtType stmtType,const ConstructNode & construct) const897 void DoForallChecker::CheckDoConcurrentExit(
898     StmtType stmtType, const ConstructNode &construct) const {
899   if (stmtType == StmtType::EXIT && ConstructIsDoConcurrent(construct)) {
900     SayBadLeave(StmtType::EXIT, "DO CONCURRENT", construct);
901   }
902 }
903 
904 // Check nesting violations for a CYCLE or EXIT statement.  Loop up the
905 // nesting levels looking for a construct that matches the CYCLE or EXIT
906 // statment.  At every construct, check for a violation.  If we find a match
907 // without finding a violation, the check is complete.
CheckNesting(StmtType stmtType,const parser::Name * stmtName) const908 void DoForallChecker::CheckNesting(
909     StmtType stmtType, const parser::Name *stmtName) const {
910   const ConstructStack &stack{context_.constructStack()};
911   for (auto iter{stack.cend()}; iter-- != stack.cbegin();) {
912     const ConstructNode &construct{*iter};
913     const std::optional<parser::Name> &constructName{
914         MaybeGetNodeName(construct)};
915     if (StmtMatchesConstruct(stmtName, stmtType, constructName, construct)) {
916       CheckDoConcurrentExit(stmtType, construct);
917       return; // We got a match, so we're finished checking
918     }
919     CheckForBadLeave(stmtType, construct);
920   }
921 
922   // We haven't found a match in the enclosing constructs
923   if (stmtType == StmtType::EXIT) {
924     context_.Say("No matching construct for EXIT statement"_err_en_US);
925   } else {
926     context_.Say("No matching DO construct for CYCLE statement"_err_en_US);
927   }
928 }
929 
930 // C1135 -- Nesting for CYCLE statements
Enter(const parser::CycleStmt & cycleStmt)931 void DoForallChecker::Enter(const parser::CycleStmt &cycleStmt) {
932   CheckNesting(StmtType::CYCLE, common::GetPtrFromOptional(cycleStmt.v));
933 }
934 
935 // C1167 and C1168 -- Nesting for EXIT statements
Enter(const parser::ExitStmt & exitStmt)936 void DoForallChecker::Enter(const parser::ExitStmt &exitStmt) {
937   CheckNesting(StmtType::EXIT, common::GetPtrFromOptional(exitStmt.v));
938 }
939 
Leave(const parser::AssignmentStmt & stmt)940 void DoForallChecker::Leave(const parser::AssignmentStmt &stmt) {
941   const auto &variable{std::get<parser::Variable>(stmt.t)};
942   context_.CheckIndexVarRedefine(variable);
943 }
944 
CheckIfArgIsDoVar(const evaluate::ActualArgument & arg,const parser::CharBlock location,SemanticsContext & context)945 static void CheckIfArgIsDoVar(const evaluate::ActualArgument &arg,
946     const parser::CharBlock location, SemanticsContext &context) {
947   common::Intent intent{arg.dummyIntent()};
948   if (intent == common::Intent::Out || intent == common::Intent::InOut) {
949     if (const SomeExpr * argExpr{arg.UnwrapExpr()}) {
950       if (const Symbol * var{evaluate::UnwrapWholeSymbolDataRef(*argExpr)}) {
951         if (intent == common::Intent::Out) {
952           context.CheckIndexVarRedefine(location, *var);
953         } else {
954           context.WarnIndexVarRedefine(location, *var); // INTENT(INOUT)
955         }
956       }
957     }
958   }
959 }
960 
961 // Check to see if a DO variable is being passed as an actual argument to a
962 // dummy argument whose intent is OUT or INOUT.  To do this, we need to find
963 // the expressions for actual arguments which contain DO variables.  We get the
964 // intents of the dummy arguments from the ProcedureRef in the "typedCall"
965 // field of the CallStmt which was filled in during expression checking.  At
966 // the same time, we need to iterate over the parser::Expr versions of the
967 // actual arguments to get their source locations of the arguments for the
968 // messages.
Leave(const parser::CallStmt & callStmt)969 void DoForallChecker::Leave(const parser::CallStmt &callStmt) {
970   if (const auto &typedCall{callStmt.typedCall}) {
971     const auto &parsedArgs{
972         std::get<std::list<parser::ActualArgSpec>>(callStmt.v.t)};
973     auto parsedArgIter{parsedArgs.begin()};
974     const evaluate::ActualArguments &checkedArgs{typedCall->arguments()};
975     for (const auto &checkedOptionalArg : checkedArgs) {
976       if (parsedArgIter == parsedArgs.end()) {
977         break; // No more parsed arguments, we're done.
978       }
979       const auto &parsedArg{std::get<parser::ActualArg>(parsedArgIter->t)};
980       ++parsedArgIter;
981       if (checkedOptionalArg) {
982         const evaluate::ActualArgument &checkedArg{*checkedOptionalArg};
983         if (const auto *parsedExpr{
984                 std::get_if<common::Indirection<parser::Expr>>(&parsedArg.u)}) {
985           CheckIfArgIsDoVar(checkedArg, parsedExpr->value().source, context_);
986         }
987       }
988     }
989   }
990 }
991 
Leave(const parser::ConnectSpec & connectSpec)992 void DoForallChecker::Leave(const parser::ConnectSpec &connectSpec) {
993   const auto *newunit{
994       std::get_if<parser::ConnectSpec::Newunit>(&connectSpec.u)};
995   if (newunit) {
996     context_.CheckIndexVarRedefine(newunit->v.thing.thing);
997   }
998 }
999 
1000 using ActualArgumentSet = std::set<evaluate::ActualArgumentRef>;
1001 
1002 struct CollectActualArgumentsHelper
1003     : public evaluate::SetTraverse<CollectActualArgumentsHelper,
1004           ActualArgumentSet> {
1005   using Base = SetTraverse<CollectActualArgumentsHelper, ActualArgumentSet>;
CollectActualArgumentsHelperFortran::semantics::CollectActualArgumentsHelper1006   CollectActualArgumentsHelper() : Base{*this} {}
1007   using Base::operator();
operator ()Fortran::semantics::CollectActualArgumentsHelper1008   ActualArgumentSet operator()(const evaluate::ActualArgument &arg) const {
1009     return Combine(ActualArgumentSet{arg},
1010         CollectActualArgumentsHelper{}(arg.UnwrapExpr()));
1011   }
1012 };
1013 
CollectActualArguments(const A & x)1014 template <typename A> ActualArgumentSet CollectActualArguments(const A &x) {
1015   return CollectActualArgumentsHelper{}(x);
1016 }
1017 
1018 template ActualArgumentSet CollectActualArguments(const SomeExpr &);
1019 
Enter(const parser::Expr & parsedExpr)1020 void DoForallChecker::Enter(const parser::Expr &parsedExpr) { ++exprDepth_; }
1021 
Leave(const parser::Expr & parsedExpr)1022 void DoForallChecker::Leave(const parser::Expr &parsedExpr) {
1023   CHECK(exprDepth_ > 0);
1024   if (--exprDepth_ == 0) { // Only check top level expressions
1025     if (const SomeExpr * expr{GetExpr(parsedExpr)}) {
1026       ActualArgumentSet argSet{CollectActualArguments(*expr)};
1027       for (const evaluate::ActualArgumentRef &argRef : argSet) {
1028         CheckIfArgIsDoVar(*argRef, parsedExpr.source, context_);
1029       }
1030     }
1031   }
1032 }
1033 
Leave(const parser::InquireSpec & inquireSpec)1034 void DoForallChecker::Leave(const parser::InquireSpec &inquireSpec) {
1035   const auto *intVar{std::get_if<parser::InquireSpec::IntVar>(&inquireSpec.u)};
1036   if (intVar) {
1037     const auto &scalar{std::get<parser::ScalarIntVariable>(intVar->t)};
1038     context_.CheckIndexVarRedefine(scalar.thing.thing);
1039   }
1040 }
1041 
Leave(const parser::IoControlSpec & ioControlSpec)1042 void DoForallChecker::Leave(const parser::IoControlSpec &ioControlSpec) {
1043   const auto *size{std::get_if<parser::IoControlSpec::Size>(&ioControlSpec.u)};
1044   if (size) {
1045     context_.CheckIndexVarRedefine(size->v.thing.thing);
1046   }
1047 }
1048 
Leave(const parser::OutputImpliedDo & outputImpliedDo)1049 void DoForallChecker::Leave(const parser::OutputImpliedDo &outputImpliedDo) {
1050   const auto &control{std::get<parser::IoImpliedDoControl>(outputImpliedDo.t)};
1051   const parser::Name &name{control.name.thing.thing};
1052   context_.CheckIndexVarRedefine(name.source, *name.symbol);
1053 }
1054 
Leave(const parser::StatVariable & statVariable)1055 void DoForallChecker::Leave(const parser::StatVariable &statVariable) {
1056   context_.CheckIndexVarRedefine(statVariable.v.thing.thing);
1057 }
1058 
1059 } // namespace Fortran::semantics
1060