Shopify / seafoam

A tool for working with compiler graphs dumped by the GraalVM compiler
MIT License
126 stars 22 forks source link

Add output formatters #48

Closed nirvdrum closed 2 years ago

nirvdrum commented 2 years ago

The primary goal of this PR is to output Seafoam's results in a machine readable format for automation use cases, while not compromising the ease-of-use afforded by the current plain text output. In order to achieve that, I'm proposing a new formatting system with plain-text and JSON formatters to start. It may very well be the case that we never need another format, but if we do, it should slot in easily enough.

chrisseaton commented 2 years ago

Idea looks good. Just needs the comments going through.