changeset 8263ac8f99d9 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=8263ac8f99d9
description:
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.
diffstat:
src/gpu-compute/gpu_dyn_inst.cc | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diffs (12 lines):
diff -r 954d3014f7f0 -r 8263ac8f99d9 src/gpu-compute/gpu_dyn_inst.cc
--- a/src/gpu-compute/gpu_dyn_inst.cc Fri May 06 15:52:34 2016 +0100
+++ b/src/gpu-compute/gpu_dyn_inst.cc Fri May 06 16:44:38 2016 -0400
@@ -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);
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
