Skip to content

[CGToDot] tool extracted from Fortran collector (#115)#119

Merged
pearzt merged 6 commits intotudasc:develfrom
gomfol12:feat/visual
Mar 4, 2026
Merged

[CGToDot] tool extracted from Fortran collector (#115)#119
pearzt merged 6 commits intotudasc:develfrom
gomfol12:feat/visual

Conversation

@gomfol12
Copy link
Contributor

Hi,
this adds a simple tool that use the DOT generator capabilities in MetaCG to generate a DOT file from a given call graph file. I primarily used it for debugging the Fortran collector. As discussed in #115 this now lives in its own PR.

Also, not sure if this should be named visual. Maybe someone has a better idea about the name?

@pearzt
Copy link
Member

pearzt commented Feb 24, 2026

How about mcg2dot for the name of the tool? (cg2dot would be closer to our current naming scheme, but tool with this name already seems to exist)

Copy link
Member

@pearzt pearzt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for extracting this into a new PR. I have only minor comments. If possible, I think it would be nice to add a few simple tests to make sure we don't break this in the future.

Copy link
Member

@jplehr jplehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for splitting this into a separate PR.
Also just minor comments.

Copy link
Member

@jplehr jplehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only missing piece IMHO is a couple of simple tests.

@gomfol12 gomfol12 changed the title visual tool extracted from Fortran collector (#115) [CGToDot] tool extracted from Fortran collector (#115) Mar 3, 2026
@pearzt pearzt merged commit 15b36cc into tudasc:devel Mar 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants