efabless / openlane2

The next generation of OpenLane, rewritten from scratch with a modular architecture
https://openlane2.readthedocs.io/
Apache License 2.0
195 stars 37 forks source link

Consistent variable naming #318

Open kareefardi opened 9 months ago

kareefardi commented 9 months ago

Description

We have some inconsistencies in variable naming. For example: PDN_CONNECT_MACROS_TO_GRID vs FP_PDN_SKIPTRIM.

EXTRA_LEFS vs EXTRA_GDS_FILES

Proposal

Be consistent with variable naming.

donn commented 9 months ago

The reason these specific variables have been renamed is that these variables are used by every OpenROAD step, not just PDN generation.

I'm open to dropping all FP_.