Updating comments and docs in response to PR comments

UoB-HPC · Dec 30, 2024 · d87cc5d · d87cc5d
1 parent e00ec65
commit d87cc5d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/docs/sphinx/user/configuring_simeng.rst b/docs/sphinx/user/configuring_simeng.rst
@@ -183,7 +183,7 @@ Num-Tage-Tables
     Only needed for a ``Tage`` predictor.  The number of tagged tables used by the predictor, in addition to a default prediction table (i.e., the BTB).  Therefore, a value of 3 for ``Num-Tage-Tables`` would result in four total prediction tables: one BTB and three tagged tables.  If no tagged tables are desired, it is recommended to use the ``GenericPredictor`` instead.
 
 Tage-Length
-    Only needed for a ``Tage`` predictor.  The number of bits used to tage the entries of the tagged tables.
+    Only needed for a ``Tage`` predictor.  The number of bits used to tag the entries of the tagged tables.
 
 .. _l1dcnf:
 

diff --git a/src/include/simeng/branchpredictors/BranchHistory.hh b/src/include/simeng/branchpredictors/BranchHistory.hh
@@ -9,7 +9,7 @@ namespace simeng {
  * access and manipulate large branch histories, as are needed in
  * sophisticated branch predictors.
  *
- * The bits of the branch history are stored in a vector of uint64_t values,
+ * The bits of the branch history are stored in an array of uint64_t values,
  * and their access/manipulation is facilitated by the public functions. */
 
 class BranchHistory {

diff --git a/src/include/simeng/branchpredictors/TagePredictor.hh b/src/include/simeng/branchpredictors/TagePredictor.hh
@@ -118,7 +118,7 @@ class TagePredictor : public BranchPredictor {
   std::vector<std::pair<uint8_t, uint64_t>> btb_;
 
   /** The bitlength of the Tagged tables' indices.
-   * Each tagged table with have 2^bits entries. */
+   * Each tagged table will have 2^bits entries. */
   uint8_t tageTableBits_;
 
   /** The number of tagged tables in the TAGE scheme.