Merge pull request #2697 from gadfort/gui-sta-endpoints

gui: add option to select just one path per endpoint in staGui

Author: gadfort

src/gui/src/staGui.cpp

@@ -1058,6 +1058,7 @@ TimingControlsDialog::TimingControlsDialog(QWidget* parent)
       path_count_spin_box_(new QSpinBox(this)),
       corner_box_(new QComboBox(this)),
       unconstrained_(new QCheckBox(this)),
+      one_path_per_endpoint_(new QCheckBox(this)),
       expand_clk_(new QCheckBox(this)),
       from_(new PinSetWidget(false, this)),
       thru_({}),
@@ -1078,6 +1079,7 @@ TimingControlsDialog::TimingControlsDialog(QWidget* parent)
 
   setUnconstrained(false);
   layout_->addRow("Unconstrained:", unconstrained_);
+  layout_->addRow("One path per endpoint:", one_path_per_endpoint_);
 
   setLayout(layout_);
 
@@ -1088,6 +1090,10 @@ TimingControlsDialog::TimingControlsDialog(QWidget* parent)
     sta_->setIncludeUnconstrainedPaths(unconstrained_->checkState()
                                        == Qt::Checked);
   });
+  connect(one_path_per_endpoint_, &QCheckBox::stateChanged, [this]() {
+    sta_->setOnePathPerEndpoint(one_path_per_endpoint_->checkState()
+                                == Qt::Checked);
+  });
 
   connect(corner_box_,
           QOverload<int>::of(&QComboBox::currentIndexChanged),
@@ -1132,6 +1138,12 @@ void TimingControlsDialog::setSTA(sta::dbSta* sta)
   to_->setSTA(sta_->getSTA());
 }
 
+void TimingControlsDialog::setOnePathPerEndpoint(bool value)
+{
+  sta_->setOnePathPerEndpoint(value);
+  one_path_per_endpoint_->setCheckState(value ? Qt::Checked : Qt::Unchecked);
+}
+
 void TimingControlsDialog::setUnconstrained(bool unconstrained)
 {
   sta_->setIncludeUnconstrainedPaths(unconstrained);

src/gui/src/staGui.h

@@ -367,6 +367,9 @@ class TimingControlsDialog : public QDialog
   void setUnconstrained(bool uncontrained);
   bool getUnconstrained() const { return sta_->isIncludeUnconstrainedPaths(); }
 
+  void setOnePathPerEndpoint(bool value);
+  bool getOnePathPerEndpoint() const { return sta_->isOnePathPerEndpoint(); }
+
   void setExpandClock(bool expand);
   bool getExpandClock() const;
 
@@ -402,6 +405,7 @@ class TimingControlsDialog : public QDialog
   QComboBox* corner_box_;
 
   QCheckBox* unconstrained_;
+  QCheckBox* one_path_per_endpoint_;
   QCheckBox* expand_clk_;
 
   PinSetWidget* from_;

src/gui/src/staGuiInterface.cpp

@@ -693,12 +693,18 @@ STAGuiInterface::STAGuiInterface(sta::dbSta* sta)
     : sta_(sta),
       corner_(nullptr),
       use_max_(true),
+      one_path_per_endpoint_(true),
       max_path_count_(1000),
       include_unconstrained_(false),
       include_capture_path_(false)
 {
 }
 
+int STAGuiInterface::getEndPointCount() const
+{
+  return sta_->endpoints()->size();
+}
+
 std::unique_ptr<TimingPathNode> STAGuiInterface::getTimingNode(
     sta::Pin* pin) const
 {
@@ -776,7 +782,7 @@ TimingPathList STAGuiInterface::getTimingPaths(
           use_max_ ? sta::MinMaxAll::max() : sta::MinMaxAll::min(),
           // group_count, endpoint_count, unique_pins
           max_path_count_,
-          max_path_count_,
+          one_path_per_endpoint_ ? 1 : max_path_count_,
           true,
           -sta::INF,
           sta::INF,  // slack_min, slack_max,

src/gui/src/staGuiInterface.h

@@ -324,6 +324,9 @@ class STAGuiInterface
     include_unconstrained_ = value;
   }
 
+  bool isOnePathPerEndpoint() const { return one_path_per_endpoint_; }
+  void setOnePathPerEndpoint(bool value) { one_path_per_endpoint_ = value; }
+
   bool isIncludeCapturePaths() const { return include_capture_path_; }
   void setIncludeCapturePaths(bool value) { include_capture_path_ = value; }
 
@@ -341,11 +344,14 @@ class STAGuiInterface
 
   std::vector<std::unique_ptr<ClockTree>> getClockTrees() const;
 
+  int getEndPointCount() const;
+
  private:
   sta::dbSta* sta_;
 
   sta::Corner* corner_;
   bool use_max_;
+  bool one_path_per_endpoint_;
   int max_path_count_;
 
   bool include_unconstrained_;

src/gui/src/timingWidget.cpp

@@ -212,6 +212,8 @@ void TimingWidget::readSettings(QSettings* settings)
 
   settings_->setPathCount(
       settings->value("path_count", settings_->getPathCount()).toInt());
+  settings_->setOnePathPerEndpoint(
+      settings->value("one_path_per_endpoint").toBool());
   settings_->setExpandClock(
       settings->value("expand_clk", settings_->getExpandClock()).toBool());
   delay_detail_splitter_->restoreState(
@@ -226,6 +228,8 @@ void TimingWidget::writeSettings(QSettings* settings)
   settings->beginGroup(objectName());
 
   settings->setValue("path_count", settings_->getPathCount());
+  settings->setValue("one_path_per_endpoint",
+                     settings_->getOnePathPerEndpoint());
   settings->setValue("expand_clk", settings_->getExpandClock());
   settings->setValue("splitter", delay_detail_splitter_->saveState());