X86: Fix a tiny typo in the load/store microop constructor.
authorGabe Black <gblack@eecs.umich.edu>
Sun, 15 Apr 2012 08:07:39 +0000 (01:07 -0700)
committerGabe Black <gblack@eecs.umich.edu>
Sun, 15 Apr 2012 08:07:39 +0000 (01:07 -0700)
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.

src/arch/x86/insts/microldstop.hh

index c618bc128a61ccde575510376996715b437e3e3d..4d96c0ec0df80798fada9d8cea6d7d0f90288d27 100644 (file)
@@ -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),