google / google/xls

[Cleanup] Refactor opt passes to use module-registry for exposing pass names

Open
#1,254 0 comments 0 reactions 0 assignees View on GitHub
cleanup estimate:M long-term-enhancement
Dominant language
C++
Stars
1.9k
Forks
283
Avg merge
2d 10h
Merged PRs (30d)
135

Description

Currently we support for `opt` passing a text pipeline which runs the given passes. This is currently manually maintained with a map from 'name -> pass' due to the fact that different passes all take different arguments.

We should refactor this so each pass library registers any passes with a single registry and has consistent arguments (or the arguments are hidden behind different types). This would simplify this code and make it easier to add new passes.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.