-
Notifications
You must be signed in to change notification settings - Fork 455
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OpenFAST Registry algorithm change: simplify USE statements #734
Conversation
This change removes USE statements for modules that are inherited through modules that are explicitly included in the Registry input files. This should reduce the memory required to build the code in certain instances, but should have no effect on the compiled code.
I did a quick memory-usage comparison using GNU Time. For OpenFAST at Here's the commands and results: OpenFAST/dev
This pull request
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Feature or improvement description
This change modifies the Registry code so that the Fortran modules it creates do not contain USE statements for modules that are inherited through other USE statements.
Impacted areas of the software
All Registry-generated *_Types.f90 files
Additional supporting information
This change cleans up the source code in modules; it is hoped that this will also require less memory to build (though this is not verified, yet).
Test results, if applicable
This should have no effect on compiled code, and thus no effect on test results.