arch: Fix the code that computes MaxMiscDestReg.
authorGabe Black <gabe.black@gmail.com>
Mon, 7 Dec 2020 12:23:39 +0000 (04:23 -0800)
committerGabe Black <gabe.black@gmail.com>
Tue, 19 Jan 2021 00:12:52 +0000 (00:12 +0000)
This code was simplified a little while ago, and the wrong variable name
was used in that computation accidentally. Fortunately the "wrong" value
would be too large, and so nothing bad would happen except a pair of
arrays would be overly large in the O3 instruction class.

Change-Id: I9694f1a8c79a62a172ef63bdd2f98fa0ace06acd
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/38383
Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Maintainer: Giacomo Travaglini <giacomo.travaglini@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/isa_parser/operand_list.py

index 12a23d9066d1b48cf1ccfc7acb8b609bfb141e0d..cea3ae4056b51fedbc6f03e07de43b64c8e244cb 100755 (executable)
@@ -135,7 +135,7 @@ class OperandList(object):
 
         parser.maxInstSrcRegs = max(parser.maxInstSrcRegs, self.numSrcRegs)
         parser.maxInstDestRegs = max(parser.maxInstDestRegs, self.numDestRegs)
-        parser.maxMiscDestRegs = max(parser.maxInstDestRegs,
+        parser.maxMiscDestRegs = max(parser.maxMiscDestRegs,
                                      self.numMiscDestRegs)
 
         # now make a final pass to finalize op_desc fields that may depend