diff options
| author | Tony Gutierrez <anthony.gutierrez@amd.com> | 2016-05-06 16:44:38 -0400 |
|---|---|---|
| committer | Tony Gutierrez <anthony.gutierrez@amd.com> | 2016-05-06 16:44:38 -0400 |
| commit | 4f3139e69647fdd67dd5604ff350e63d38f9306f (patch) | |
| tree | 61d1d00e642549c46684d869475415aaf14e9b7b | |
| parent | 84cfa10b15cd472d55b5f51e656da0473199cf60 (diff) | |
| download | gem5-4f3139e69647fdd67dd5604ff350e63d38f9306f.tar.xz | |
gpu-compute: fix uninitialized member bug in GPUDynInst
the n_reg field in the GPUDynInst is not currently set in the constructor.
if it is not set externally, there are assertion failures that may occur
if the random value it gets is just right. here we set it to 0 by default.
| -rw-r--r-- | src/gpu-compute/gpu_dyn_inst.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gpu-compute/gpu_dyn_inst.cc b/src/gpu-compute/gpu_dyn_inst.cc index 83e348dbe..594e45907 100644 --- a/src/gpu-compute/gpu_dyn_inst.cc +++ b/src/gpu-compute/gpu_dyn_inst.cc @@ -43,7 +43,7 @@ GPUDynInst::GPUDynInst(ComputeUnit *_cu, Wavefront *_wf, GPUStaticInst *_staticInst, uint64_t instSeqNum) : GPUExecContext(_cu, _wf), m_op(Enums::MO_UNDEF), - memoryOrder(Enums::MEMORY_ORDER_NONE), useContinuation(false), + memoryOrder(Enums::MEMORY_ORDER_NONE), n_reg(0) ,useContinuation(false), statusBitVector(0), staticInst(_staticInst), _seqNum(instSeqNum) { tlbHitLevel.assign(VSZ, -1); |