mpl: update cluster types

[joaomai]

Description

Clusters can be of 3 types nowadays (standard cell, mixed and hard macro) while having many isSomething methods to cover IOs, pins and specialized macro structures.

  bool is_cluster_of_unplaced_io_pins_{false};
  bool is_cluster_of_unconstrained_io_pins_{false};
  bool is_io_pad_cluster_{false};
  bool is_io_bundle_{false};
  bool is_array_of_interconnected_macros_{false};
  bool is_macro_array_{false};

Suggested Solution

Consolidate the many implicit cluster types under the ClusterType enum instad of relying on isSomething methods.

Additional Context

No response

[AcKoucher]

Suggestions for the work here:

• Type should be a member of Cluster;
• The types should be at least:
  • Macro
  • StdCell
  • Mixed
  • ConstrainedIOs
  • UnconstrainedIOs
  • IOBundle
  • PAD

[maliberty]

What is the difference between IOBundle and *IOs?

[AcKoucher]

What is the difference between IOBundle and *IOs?

IOBundle is the abstraction for IOs that are already placed. We divide each edge of the core in 5 segments and create IOBundles for the segments with placed IOs.

ConstrainedIOs and UnconstrainedIOs are abstraction for groups of pins that are not placed. They're grouped by constraint i.e., pins with the same constraints belong to the same cluster.

[alokkumardalei-wq]

Hello @maliberty , @joaomai and @AcKoucher is anyone already working on it if not please assign this to me, would happy to work on this.
Thank you.

[AcKoucher]

Hi @alokkumardalei-wq! I'm glad to see you're interested on working on this issue. Afaik this is low priority and you could tackle it. I'll wait for @joaomai 's take as he is the one who tracked the work.

[alokkumardalei-wq]

Thanks @AcKoucher happy to hear that , but l will wait for @joaomai confirmation , in the mean time I would be familiarizing myself with the code before making any changes to it.

Thank you!

[joaomai]

Hello @alokkumardalei-wq, go ahead! The issue description is really succinct, so feel free to ask questions if something is unclear.

[alokkumardalei-wq]

Hello @joaomai and @AcKoucher , I have raised for this , please have look onto this when you have time and I would be happy to address any further feedback.

Thank you.