circt icon indicating copy to clipboard operation
circt copied to clipboard

[FIRRTL] Update rationale mentioning use of IntegerAttr for port directions

Open dtzSiFive opened this issue 7 months ago • 0 comments

As of #6875, port directions as now encoded with a DenseArray.

In the rationale we mention we still encode with IntegerAttr, this should be updated to reflect what's been learned and changed.

https://circt.llvm.org/docs/Dialects/FIRRTL/RationaleFIRRTL/#input-and-output-module-ports .

dtzSiFive avatar Jun 26 '24 23:06 dtzSiFive