-
Notifications
You must be signed in to change notification settings - Fork 299
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
Move to new pass generation approach #3962
Comments
uenoku
added a commit
that referenced
this issue
Nov 15, 2022
This applies a new pass generation method(ref:#3962) to seq dialect passes, that automatically defines a struct of options (e.g. `LowerSeqFIRRTLToSVOptions`) and creates a pass constructor to take the option.
This was referenced Jun 13, 2024
fzi-hielscher
added a commit
that referenced
this issue
Jun 14, 2024
Refactor includes of TableGen passes in the "Conversion" sub-tree to the new style. See #3962 , https://reviews.llvm.org/D143773
This was referenced Jun 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Recently, table-gen was changed to split the emissions of passes into different chunks: https://reviews.llvm.org/D131839
This change not only splits a large monolith into smaller pieces but also adds support for generating pass constructors that expect pass options. The latter will be useful for tools like
hlstool
as many passes do currently not have constructures that take their options.Link to the PSA: https://discourse.llvm.org/t/psa-deprecation-warning-per-pass-autogenerated-macros-and-pass-options/64998
The text was updated successfully, but these errors were encountered: