Unverified Commit 610f595e authored by mergify[bot]'s avatar mergify[bot] Committed by GitHub
Browse files

Generalize keyword collision to name manipulation, Add {Lower,Upper}CaseNames...


Generalize keyword collision to name manipulation, Add {Lower,Upper}CaseNames Transforms (bp #1592) (#1612)

* Refactor RemoveKeywordCollisions->ManipulateNames

Generalize the operations of the RemoveKeywordCollisions transform
into a new ManipulateNames transform. The ManipulateNames transform is
an abstract transform for making conditional modifications to
keywords/names in a FIRRTL circuit.

Signed-off-by: default avatarSchuyler Eldridge <schuyler.eldridge@ibm.com>
(cherry picked from commit c534c5ab

)

* Add features.{LowerCaseNames, UpperCaseNames} transforms

Creates the features package and populates it with two new transforms:
LowerCaseNames and UpperCaseNames. These transforms convert all names
in a FIRRTL circuit to lower case or upper case. This is intended to
help align generated Verilog with the policies of the
company/institution using it.

Signed-off-by: default avatarSchuyler Eldridge <schuyler.eldridge@ibm.com>

squash! Add LowerCaseNames and UpperCaseNames transforms

(cherry picked from commit c8dcdacf

)

* Add --change-name-case <lower|upper> option

Adds an options to the FIRRTL compiler command line to schedule the
LowerCaseNames and UpperCaseNames transforms.

Signed-off-by: default avatarSchuyler Eldridge <schuyler.eldridge@ibm.com>
(cherry picked from commit d3ab7e2d

)

* Add test of {Lower, Upper}CaseNames

Signed-off-by: default avatarSchuyler Eldridge <schuyler.eldridge@ibm.com>
(cherry picked from commit 93c078b8

)

Co-authored-by: default avatarSchuyler Eldridge <schuyler.eldridge@ibm.com>
Co-authored-by: default avatarmergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
parent 83c36dbc
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment