Chun-Chen TK Hsu has uploaded this change for review. ( https://gem5-review.googlesource.com/9121


Change subject: arm: Fix maybe-uninitialized GCC warnings
......................................................................

arm: Fix maybe-uninitialized GCC warnings

GCC 7 generates maybe-uninitialized warnings at the code that updates
the "dest" variables in the writeVecElem function of neon64_mem.hh file.
It is because the generated code does not appropriately initialize the
output variable before passing it to the writeVecElem function.  This
patch initializes the output variable to fix this.

Change-Id: I50a8f4e456ccdcaa3db1392ec097017450c56ecb
Signed-off-by: Chun-Chen Hsu <chunchen...@google.com>
---
M src/arch/arm/isa/insts/neon64_mem.isa
1 file changed, 2 insertions(+), 1 deletion(-)



diff --git a/src/arch/arm/isa/insts/neon64_mem.isa b/src/arch/arm/isa/insts/neon64_mem.isa
index 4d32412..4511ad1 100644
--- a/src/arch/arm/isa/insts/neon64_mem.isa
+++ b/src/arch/arm/isa/insts/neon64_mem.isa
@@ -336,7 +336,8 @@
             eCode = '''
                 //input data from scratch area
                 VReg input[4] = { {0, 0}, {0, 0}, {0, 0}, {0, 0} };
-                VReg output[2];  //output data to arch. SIMD regs
+                //output data to arch. SIMD regs
+                VReg output[2] = { {0, 0}, {0, 0} };
             '''

             eCode += getInputCodeOp1L

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

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I50a8f4e456ccdcaa3db1392ec097017450c56ecb
Gerrit-Change-Number: 9121
Gerrit-PatchSet: 1
Gerrit-Owner: Chun-Chen TK Hsu <chunchen...@google.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to