Moved UI function code to lambdas & Fixed BTB having incorrect number…

… of bits
cvut · Jul 8, 2024 · e54b47d · e54b47d
1 parent 705897d
commit e54b47d
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 47 deletions.
diff --git a/src/gui/dialogs/new/newdialog.cpp b/src/gui/dialogs/new/newdialog.cpp
@@ -130,24 +130,30 @@ NewDialog::NewDialog(QWidget *parent, QSettings *settings) : QDialog(parent) {
     connect(ui->osemu_fs_root, &QLineEdit::textChanged, this, &NewDialog::osemu_fs_root_change);
 
     // Branch predictor
-    connect(
-        ui->group_branch_predictor, QOverload<bool>::of(&QGroupBox::toggled), this,
-        &NewDialog::bp_enabled_change);
+    connect(ui->group_branch_predictor, QOverload<bool>::of(&QGroupBox::toggled), this, [this] {
+        config->set_bp_enabled(ui->group_branch_predictor->isChecked());
+    });
     connect(
         ui->select_bp_type, QOverload<int>::of(&QComboBox::activated), this,
         &NewDialog::bp_type_change);
-    connect(
-        ui->select_bp_init_state, QOverload<int>::of(&QComboBox::activated), this,
-        &NewDialog::bp_init_state_change);
-    connect(
-        ui->slider_bp_btb_bits, &QAbstractSlider::valueChanged, this,
-        &NewDialog::bp_btb_bits_change);
-    connect(
-        ui->slider_bp_bhr_bits, &QAbstractSlider::valueChanged, this,
-        &NewDialog::bp_bhr_bits_change);
-    connect(
-        ui->slider_bp_bht_addr_bits, &QAbstractSlider::valueChanged, this,
-        &NewDialog::bp_bht_addr_bits_change);
+    connect(ui->select_bp_init_state, QOverload<int>::of(&QComboBox::activated), this, [this] {
+        config->set_bp_init_state(
+            ui->select_bp_init_state->currentData().value<machine::PredictorState>());
+    });
+    connect(ui->slider_bp_btb_bits, &QAbstractSlider::valueChanged, this, [this] {
+        config->set_bp_btb_bits((uint8_t)ui->slider_bp_btb_bits->value());
+        ui->text_bp_btb_bits_number->setText(QString::number(config->get_bp_btb_bits()));
+    });
+    connect(ui->slider_bp_bhr_bits, &QAbstractSlider::valueChanged, this, [this] {
+        config->set_bp_bhr_bits((uint8_t)ui->slider_bp_bhr_bits->value());
+        ui->text_bp_bhr_bits_number->setText(QString::number(config->get_bp_bhr_bits()));
+        ui->text_bp_bht_bits_number->setText(QString::number(config->get_bp_bht_bits()));
+    });
+    connect(ui->slider_bp_bht_addr_bits, &QAbstractSlider::valueChanged, this, [this] {
+        config->set_bp_bht_addr_bits((uint8_t)ui->slider_bp_bht_addr_bits->value());
+        ui->text_bp_bht_addr_bits_number->setText(QString::number(config->get_bp_bht_addr_bits()));
+        ui->text_bp_bht_bits_number->setText(QString::number(config->get_bp_bht_bits()));
+    });
 
     cache_handler_d = new NewDialogCacheHandler(this, ui_cache_d.data());
     cache_handler_p = new NewDialogCacheHandler(this, ui_cache_p.data());
@@ -385,10 +391,6 @@ void NewDialog::reset_at_compile_change(bool v) {
     config->set_reset_at_compile(v);
 }
 
-void NewDialog::bp_enabled_change() {
-    config->set_bp_enabled(ui->group_branch_predictor->isChecked());
-}
-
 void NewDialog::bp_type_change() {
     // Read branch predictor type from GUI and store it in the config
     const machine::PredictorType predictor_type {
@@ -458,28 +460,6 @@ void NewDialog::bp_type_change() {
     }
 }
 
-void NewDialog::bp_init_state_change() {
-    config->set_bp_init_state(
-        ui->select_bp_init_state->currentData().value<machine::PredictorState>());
-}
-
-void NewDialog::bp_btb_bits_change() {
-    config->set_bp_btb_bits((uint8_t)ui->slider_bp_btb_bits->value());
-    ui->text_bp_btb_bits_number->setText(QString::number(config->get_bp_btb_bits()));
-}
-
-void NewDialog::bp_bhr_bits_change() {
-    config->set_bp_bhr_bits((uint8_t)ui->slider_bp_bhr_bits->value());
-    ui->text_bp_bhr_bits_number->setText(QString::number(config->get_bp_bhr_bits()));
-    ui->text_bp_bht_bits_number->setText(QString::number(config->get_bp_bht_bits()));
-}
-
-void NewDialog::bp_bht_addr_bits_change() {
-    config->set_bp_bht_addr_bits((uint8_t)ui->slider_bp_bht_addr_bits->value());
-    ui->text_bp_bht_addr_bits_number->setText(QString::number(config->get_bp_bht_addr_bits()));
-    ui->text_bp_bht_bits_number->setText(QString::number(config->get_bp_bht_bits()));
-}
-
 void NewDialog::bp_toggle_history_table_ui(bool enabled) {
     ui->select_bp_init_state->setEnabled(enabled);
     ui->slider_bp_bhr_bits->setEnabled(enabled);

diff --git a/src/gui/dialogs/new/newdialog.h b/src/gui/dialogs/new/newdialog.h
@@ -55,12 +55,7 @@ private slots:
     void reset_at_compile_change(bool);
 
     // Branch Predictor
-    void bp_enabled_change();
     void bp_type_change();
-    void bp_init_state_change();
-    void bp_btb_bits_change();
-    void bp_bhr_bits_change();
-    void bp_bht_addr_bits_change();
     void bp_toggle_history_table_ui(bool enabled);
 
 private:

diff --git a/src/machine/predictor.cpp b/src/machine/predictor.cpp
@@ -614,7 +614,7 @@ BranchPredictor::BranchPredictor(
     }
 
     bhr = new BranchHistoryRegister(number_of_bhr_bits);
-    btb = new BranchTargetBuffer(number_of_bht_bits);
+    btb = new BranchTargetBuffer(number_of_btb_bits);
 
     if (enabled) {
         connect(