base: Fix uninitialized variable in Flag
This was uninitialized, and was breaking expected values under certain situations. Change-Id: If51ab6ae038c7c397bc83de1c73af348c1db4ef8 Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br> Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/38707 Reviewed-by: Bobby R. Bruce <bbruce@ucdavis.edu> Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com> Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu> Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
committed by
Daniel Carvalho
parent
3b03eaab9c
commit
ba284a13dd
@@ -100,6 +100,8 @@ Flag::Flag(const char *name, const char *desc)
|
||||
panic("Flag %s already defined!", name);
|
||||
|
||||
++allFlagsVersion;
|
||||
|
||||
sync();
|
||||
}
|
||||
|
||||
Flag::~Flag()
|
||||
|
||||
@@ -82,15 +82,13 @@ class Flag
|
||||
class SimpleFlag : public Flag
|
||||
{
|
||||
protected:
|
||||
bool _tracing; // tracing is enabled and flag is on
|
||||
bool _status; // flag status
|
||||
bool _tracing = false; // tracing is enabled and flag is on
|
||||
bool _status = false; // flag status
|
||||
|
||||
void sync() override { _tracing = _globalEnable && _status; }
|
||||
|
||||
public:
|
||||
SimpleFlag(const char *name, const char *desc)
|
||||
: Flag(name, desc), _status(false)
|
||||
{ }
|
||||
SimpleFlag(const char *name, const char *desc) : Flag(name, desc) {}
|
||||
|
||||
bool status() const override { return _tracing; }
|
||||
|
||||
|
||||
Reference in New Issue
Block a user