Branch: refs/heads/main
Revision: 6516dff
Author: mppf
Link: dyno: avoid recursive queries in module resolution by mppf · Pull Request #19757 · chapel-lang/chapel · GitHub
Log Message:
Merge pull request #19757 from mppf/remove-recursive-queries-resolve-module
dyno: avoid recursive queries in module resolution
For https://github.com/Cray/chapel-private/issues/3290
This PR updates the module resolution process to have a query that
resolves each module init statement individually rather than using a
partial result for resolving the whole module. This fixes a crash with a
simple pattern in incremental compilation that is added as a test.
While there, this PR removes a bunch of noise from the query trace
output. It could still be improved further.
Reviewed by @dlongnecke-cray - thanks!
Modified Files:
M compiler/dyno/include/chpl/queries/ID.h
M compiler/dyno/include/chpl/queries/query-impl.h
M compiler/dyno/include/chpl/resolution/resolution-queries.h
M compiler/dyno/lib/queries/Context.cpp
M compiler/dyno/lib/resolution/Resolver.cpp
M compiler/dyno/lib/resolution/Resolver.h
M compiler/dyno/lib/resolution/resolution-queries.cpp
M compiler/dyno/lib/uast/AstNode.cpp
M compiler/dyno/test/resolution/testInteractive.cpp
M compiler/dyno/test/resolution/testResolve.cpp
Compare: https://github.com/chapel-lang/chapel/compare/be2f9aa65573...6516dff48603