From: Gabe Black <gblack@eecs.umich.edu>
Date: Sun, 15 Apr 2012 08:07:39 +0000 (-0700)
Subject: X86: Fix a tiny typo in the load/store microop constructor.
X-Git-Tag: stable_2012_06_28~119
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8fe112d61b08b83b6b7f27daa9daaa521e67deaf;p=gem5.git

X86: Fix a tiny typo in the load/store microop constructor.

The parameter is _machInst, which is very similar to the member machInst. If
machInst is used to pass the parameter to a lower level constructor, what
really happens is that machInst is set to whatever it already happened to be,
effectively leaving it uninitialized.
---

diff --git a/src/arch/x86/insts/microldstop.hh b/src/arch/x86/insts/microldstop.hh
index c618bc128..4d96c0ec0 100644
--- a/src/arch/x86/insts/microldstop.hh
+++ b/src/arch/x86/insts/microldstop.hh
@@ -82,7 +82,7 @@ namespace X86ISA
                 uint8_t _dataSize, uint8_t _addressSize,
                 Request::FlagsType _memFlags,
                 OpClass __opClass) :
-        X86MicroopBase(machInst, mnem, _instMnem, setFlags, __opClass),
+        X86MicroopBase(_machInst, mnem, _instMnem, setFlags, __opClass),
                 scale(_scale), index(_index.idx), base(_base.idx),
                 disp(_disp), segment(_segment.idx),
                 data(_data.idx),