Giacomo Travaglini has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44505 )

Change subject: misc: Fix VecPredReg coding style
......................................................................

misc: Fix VecPredReg coding style

* get_bits -> getBits
* set_bits -> setBits
* set_raw -> setRaw
* get_raw -> getRaw

Change-Id: I57c0217dc399b7e1c5b007ed862d7ed221d5ac0b
Signed-off-by: Giacomo Travaglini <[email protected]>
---
M src/arch/arm/fastmodel/iris/thread_context.cc
M src/arch/arm/isa/insts/sve.isa
M src/arch/generic/vec_pred_reg.hh
3 files changed, 12 insertions(+), 12 deletions(-)



diff --git a/src/arch/arm/fastmodel/iris/thread_context.cc b/src/arch/arm/fastmodel/iris/thread_context.cc
index 9567b11..b2173e1 100644
--- a/src/arch/arm/fastmodel/iris/thread_context.cc
+++ b/src/arch/arm/fastmodel/iris/thread_context.cc
@@ -693,13 +693,13 @@
     size_t num_bits = reg.NUM_BITS;
     uint8_t *bytes = (uint8_t *)result.data.data();
     while (num_bits > 8) {
-        reg.set_bits(offset, 8, *bytes);
+        reg.setBits(offset, 8, *bytes);
         offset += 8;
         num_bits -= 8;
         bytes++;
     }
     if (num_bits)
-        reg.set_bits(offset, num_bits, *bytes);
+        reg.setBits(offset, num_bits, *bytes);

     return reg;
 }
diff --git a/src/arch/arm/isa/insts/sve.isa b/src/arch/arm/isa/insts/sve.isa
index 03775ca..8a4e724 100644
--- a/src/arch/arm/isa/insts/sve.isa
+++ b/src/arch/arm/isa/insts/sve.isa
@@ -2623,7 +2623,7 @@
                 code +='''
             const SElement& srcElem1 = auxPOp1[i];'''
             code += '''
-            destPred.set_raw(i, 0);
+            destPred.setRaw(i, 0);
             PDest_xd[i] = srcElem1;'''
         else:
             if unpackHalf == Unpack.High:
@@ -2836,8 +2836,8 @@
         ArmISA::VecPredRegContainer tmpPredC;
         auto auxPOp1 = tmpPredC.as<Element>();
         for (unsigned i = 0; i < eCount; ++i) {
-            uint8_t v = POp1_x.get_raw(i);
-            auxPOp1.set_raw(i, v);
+            uint8_t v = POp1_x.getRaw(i);
+            auxPOp1.setRaw(i, v);
         }
         PDest_x[0] = 0;'''
         else:
@@ -2854,7 +2854,7 @@
             AA64FpDest_x[i] = auxOp1[eCount - i - 1];'''
         else:
             code += '''
-            destPred.set_raw(i, auxPOp1.get_raw(eCount - i - 1));'''
+            destPred.setRaw(i, auxPOp1.getRaw(eCount - i - 1));'''
         code += '''
         }'''
         iop = ArmInstObjParams(name, 'Sve' + Name, 'SveUnaryUnpredOp',
diff --git a/src/arch/generic/vec_pred_reg.hh b/src/arch/generic/vec_pred_reg.hh
index b238357..46f6f2f 100644
--- a/src/arch/generic/vec_pred_reg.hh
+++ b/src/arch/generic/vec_pred_reg.hh
@@ -118,18 +118,18 @@
     /// Return an element of the predicate register as it appears
     /// in the raw (untyped) internal representation
     uint8_t
-    get_raw(size_t idx) const
+    getRaw(size_t idx) const
     {
-        return container.get_bits(idx * (Packed ? 1 : sizeof(VecElem)),
+        return container.getBits(idx * (Packed ? 1 : sizeof(VecElem)),
                 (Packed ? 1 : sizeof(VecElem)));
     }

     /// Write a raw value in an element of the predicate register
     template<bool Condition = !Const>
     std::enable_if_t<Condition>
-    set_raw(size_t idx, uint8_t val)
+    setRaw(size_t idx, uint8_t val)
     {
-        container.set_bits(idx * (Packed ? 1 : sizeof(VecElem)),
+        container.setBits(idx * (Packed ? 1 : sizeof(VecElem)),
                 (Packed ? 1 : sizeof(VecElem)), val);
     }

@@ -302,7 +302,7 @@
     /// Returns a subset of bits starting from a specific element in the
     /// container.
     uint8_t
-    get_bits(size_t idx, uint8_t nbits) const
+    getBits(size_t idx, uint8_t nbits) const
     {
         assert(nbits > 0 && nbits <= 8 && (idx + nbits - 1) < NumBits);
         uint8_t v = 0;
@@ -317,7 +317,7 @@
     /// Set a subset of bits starting from a specific element in the
     /// container.
     void
-    set_bits(size_t idx, uint8_t nbits, uint8_t bval)
+    setBits(size_t idx, uint8_t nbits, uint8_t bval)
     {
         assert(nbits > 0 && nbits <= 8 && (idx + nbits - 1) < NumBits);
         for (int i = 0; i < nbits; ++i, ++idx) {

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44505
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I57c0217dc399b7e1c5b007ed862d7ed221d5ac0b
Gerrit-Change-Number: 44505
Gerrit-PatchSet: 1
Gerrit-Owner: Giacomo Travaglini <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to