OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

drt: pin access split

Open bnmfw opened this issue 1 year ago • 0 comments

This PR is waiting for every PA PR to merge, #5921, #5903 and #5891.

This PR creates three new PA files, one for each of the FlexPA_prep.cpp responsibilities:

  • Generating Access Points for Unique Instances: FlexPA_acc_point.cpp
  • Generating Access Patterns for Unique Instances: FlexPA_acc_patterns.cpp
  • Generating Access Patterns to Rows of Instances: FlexPA_row_patterns.cpp

Some functions that couldn't be easily separated were kept in FlexPA_prep.cpp, this might be due to being dependant to more than one of the new files or more than one of the new files depending on it.

There might be a case for moving everything in FlexPA_prep.cpp to 'FlexPA.cpp' as now there are few functions left.

I copied imports and copyright to new files. I commented any import that did not stop the project from compiling and running.

FlexPA_acc_point.cpp is still big 1483 LOC so it might have to be split even further.

bnmfw avatar Oct 14 '24 19:10 bnmfw