[Chapel Merge] dyno: optionally resolve and convert types in conv

Notifications Commits
1

Branch: refs/heads/main
Revision: 4cf4575
Author: mppf
Link: dyno: optionally resolve and convert types in converter by mppf · Pull Request #19822 · chapel-lang/chapel · GitHub
Log Message:

Merge pull request #19822 from mppf/dyno-convert-types

dyno: optionally resolve and convert types in converter

This PR enables the conversion process in convert-uast.cpp to:

  • resolve the module to compute types etc
  • convert types and params from the resolution result to the production AST
  • set the types and param values when the resolution result is known

The optional resolution process within convert-uast.cpp is controlled by shouldResolve which currently just returns false.

To do so, I refactored a couple of functions in build.cpp so that the functionality could be more easily reused when including types in the conversion process.

Reviewed by @arezaii - thanks!