Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/49706 )
Change subject: sim: Fix style in insttracer.hh.
......................................................................
sim: Fix style in insttracer.hh.
Change-Id: Iddf032ae03ef20d6220c298424779dad726f5179
---
M src/sim/insttracer.hh
1 file changed, 65 insertions(+), 33 deletions(-)
diff --git a/src/sim/insttracer.hh b/src/sim/insttracer.hh
index 58c5fb1..6bb5bd3 100644
--- a/src/sim/insttracer.hh
+++ b/src/sim/insttracer.hh
@@ -79,9 +79,9 @@
* Memory request information in the instruction accessed memory.
* @see mem_valid
*/
- Addr addr; ///< The address that was accessed
- Addr size; ///< The size of the memory request
- unsigned flags; ///< The flags that were assigned to the request.
+ Addr addr = 0; ///< The address that was accessed
+ Addr size = 0; ///< The size of the memory request
+ unsigned flags = 0; ///< The flags that were assigned to the request.
/** @} */
@@ -105,13 +105,13 @@
* This records the serial number that the instruction was fetched in.
* @see fetch_seq_valid
*/
- InstSeqNum fetch_seq;
+ InstSeqNum fetch_seq = 0;
/** @defgroup commit_seq
* This records the instruction number that was committed in the
pipeline
* @see cp_seq_valid
*/
- InstSeqNum cp_seq;
+ InstSeqNum cp_seq = 0;
/** @ingroup data
* What size of data was written?
@@ -126,43 +126,40 @@
DataDouble = 3,
DataVec = 5,
DataVecPred = 6
- } data_status;
+ } data_status = DataInvalid;
/** @ingroup memory
* Are the memory fields in the record valid?
*/
- bool mem_valid;
+ bool mem_valid = false;
/** @ingroup fetch_seq
* Are the fetch sequence number fields valid?
*/
- bool fetch_seq_valid;
+ bool fetch_seq_valid = false;
/** @ingroup commit_seq
* Are the commit sequence number fields valid?
*/
- bool cp_seq_valid;
+ bool cp_seq_valid = false;
/** is the predicate for execution this inst true or false (not
execed)?
*/
- bool predicate;
+ bool predicate = true;
/**
* Did the execution of this instruction fault? (requires ExecFaulting
* to be enabled)
*/
- bool faulting;
+ bool faulting = false;
public:
InstRecord(Tick _when, ThreadContext *_thread,
const StaticInstPtr _staticInst,
TheISA::PCState _pc,
- const StaticInstPtr _macroStaticInst = NULL)
+ const StaticInstPtr _macroStaticInst=nullptr)
: when(_when), thread(_thread), staticInst(_staticInst), pc(_pc),
- macroStaticInst(_macroStaticInst), addr(0), size(0), flags(0),
- fetch_seq(0), cp_seq(0), data_status(DataInvalid),
mem_valid(false),
- fetch_seq_valid(false), cp_seq_valid(false), predicate(true),
- faulting(false)
- { }
+ macroStaticInst(_macroStaticInst)
+ {}
virtual ~InstRecord()
{
@@ -176,9 +173,13 @@
}
void setWhen(Tick new_when) { when = new_when; }
- void setMem(Addr a, Addr s, unsigned f)
+ void
+ setMem(Addr a, Addr s, unsigned f)
{
- addr = a; size = s; flags = f; mem_valid = true;
+ addr = a;
+ size = s;
+ flags = f;
+ mem_valid = true;
}
template <typename T, size_t N>
@@ -192,17 +193,42 @@
"Type T has an unrecognized size.");
}
- void setData(uint64_t d) { data.as_int = d; data_status = DataInt64; }
- void setData(uint32_t d) { data.as_int = d; data_status = DataInt32; }
- void setData(uint16_t d) { data.as_int = d; data_status = DataInt16; }
- void setData(uint8_t d) { data.as_int = d; data_status = DataInt8; }
+ void
+ setData(uint64_t d)
+ {
+ data.as_int = d;
+ data_status = DataInt64;
+ }
+ void
+ setData(uint32_t d)
+ {
+ data.as_int = d;
+ data_status = DataInt32;
+ }
+ void
+ setData(uint16_t d)
+ {
+ data.as_int = d;
+ data_status = DataInt16;
+ }
+ void
+ setData(uint8_t d)
+ {
+ data.as_int = d;
+ data_status = DataInt8;
+ }
void setData(int64_t d) { setData((uint64_t)d); }
void setData(int32_t d) { setData((uint32_t)d); }
void setData(int16_t d) { setData((uint16_t)d); }
void setData(int8_t d) { setData((uint8_t)d); }
- void setData(double d) { data.as_double = d; data_status = DataDouble;
}
+ void
+ setData(double d)
+ {
+ data.as_double = d;
+ data_status = DataDouble;
+ }
void
setData(TheISA::VecRegContainer& d)
@@ -218,11 +244,19 @@
data_status = DataVecPred;
}
- void setFetchSeq(InstSeqNum seq)
- { fetch_seq = seq; fetch_seq_valid = true; }
+ void
+ setFetchSeq(InstSeqNum seq)
+ {
+ fetch_seq = seq;
+ fetch_seq_valid = true;
+ }
- void setCPSeq(InstSeqNum seq)
- { cp_seq = seq; cp_seq_valid = true; }
+ void
+ setCPSeq(InstSeqNum seq)
+ {
+ cp_seq = seq;
+ cp_seq_valid = true;
+ }
void setPredicate(bool val) { predicate = val; }
@@ -258,16 +292,14 @@
class InstTracer : public SimObject
{
public:
- InstTracer(const Params &p) : SimObject(p)
- {}
+ InstTracer(const Params &p) : SimObject(p) {}
- virtual ~InstTracer()
- {};
+ virtual ~InstTracer() {}
virtual InstRecord *
getInstRecord(Tick when, ThreadContext *tc,
const StaticInstPtr staticInst, TheISA::PCState pc,
- const StaticInstPtr macroStaticInst = NULL) = 0;
+ const StaticInstPtr macroStaticInst=nullptr) = 0;
};
} // namespace Trace
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/49706
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: Iddf032ae03ef20d6220c298424779dad726f5179
Gerrit-Change-Number: 49706
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[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
