-
- Downloads
[Flang][OpenMP][Sema] Support propagation of REQUIRES information across program units
This patch adds support for storing OpenMP REQUIRES information in the semantics symbols for programs/subprograms and modules/submodules, and populates them during directive resolution. A pass is added to name resolution that makes sure this information is also propagated across top-level programs, functions and subprograms. Storing REQUIRES information inside of semantics symbols will also allow supporting the propagation of this information across Fortran modules. This will come as a separate patch. The `bool DirectiveAttributeVisitor::Pre(const parser::SpecificationPart &x)` method is removed since it resulted in specification parts being visited twice. This is patch 3/5 of a series splitting D149337 to simplify review. Differential Revision: https://reviews.llvm.org/D157983
Showing
- flang/examples/FeatureList/FeatureList.cpp 2 additions, 1 deletionflang/examples/FeatureList/FeatureList.cpp
- flang/include/flang/Common/Fortran.h 3 additions, 0 deletionsflang/include/flang/Common/Fortran.h
- flang/include/flang/Parser/dump-parse-tree.h 1 addition, 1 deletionflang/include/flang/Parser/dump-parse-tree.h
- flang/include/flang/Parser/parse-tree.h 2 additions, 2 deletionsflang/include/flang/Parser/parse-tree.h
- flang/include/flang/Semantics/symbol.h 33 additions, 3 deletionsflang/include/flang/Semantics/symbol.h
- flang/lib/Parser/openmp-parsers.cpp 3 additions, 3 deletionsflang/lib/Parser/openmp-parsers.cpp
- flang/lib/Parser/unparse.cpp 1 addition, 11 deletionsflang/lib/Parser/unparse.cpp
- flang/lib/Semantics/resolve-directives.cpp 152 additions, 5 deletionsflang/lib/Semantics/resolve-directives.cpp
- flang/lib/Semantics/resolve-directives.h 2 additions, 0 deletionsflang/lib/Semantics/resolve-directives.h
- flang/lib/Semantics/resolve-names.cpp 4 additions, 1 deletionflang/lib/Semantics/resolve-names.cpp
- flang/test/Semantics/OpenMP/requires09.f90 14 additions, 0 deletionsflang/test/Semantics/OpenMP/requires09.f90
Loading
Please register or sign in to comment