summaryrefslogtreecommitdiff
path: root/src/arch/power
diff options
context:
space:
mode:
authorSteve Reinhardt <steve.reinhardt@amd.com>2016-02-06 17:21:20 -0800
committerSteve Reinhardt <steve.reinhardt@amd.com>2016-02-06 17:21:20 -0800
commitf5343df1e1999e66c39b95085b77b547e7d94262 (patch)
tree87c16b2fd80e22d4402833604182bd3f72411464 /src/arch/power
parent92b750d5efb56359e3c95ca6bc093ab4b6395aa1 (diff)
downloadgem5-f5343df1e1999e66c39b95085b77b547e7d94262.tar.xz
arch: get rid of dummy var init
MemOperand variables were being initialized to 0 "to avoid 'uninitialized variable' errors" but these no longer seem to be a problem (with the exception of one use case in POWER that is arguably broken and easily fixed here). Getting rid of the initialization is necessary to set up a subsequent patch which extends memory operands to possibly not be scalars, making the '= 0' initialization no longer feasible.
Diffstat (limited to 'src/arch/power')
-rw-r--r--src/arch/power/isa/decoder.isa2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/arch/power/isa/decoder.isa b/src/arch/power/isa/decoder.isa
index 11d222390..30002fe33 100644
--- a/src/arch/power/isa/decoder.isa
+++ b/src/arch/power/isa/decoder.isa
@@ -309,10 +309,10 @@ decode OPCODE default Unknown::unknown() {
151: stwx({{ Mem = Rs; }});
150: stwcx({{
bool store_performed = false;
+ Mem = Rs;
if (Rsv) {
if (RsvLen == 4) {
if (RsvAddr == EA) {
- Mem = Rs;
store_performed = true;
}
}