From: Gabe Black <gabe.black@gmail.com>
Date: Mon, 7 Dec 2020 12:23:39 +0000 (-0800)
Subject: arch: Fix the code that computes MaxMiscDestReg.
X-Git-Tag: develop-gem5-snapshot~278
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d7083448e79e446b52b42b9dbf31d49e3ac506ff;p=gem5.git

arch: Fix the code that computes MaxMiscDestReg.

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>
---

diff --git a/src/arch/isa_parser/operand_list.py b/src/arch/isa_parser/operand_list.py
index 12a23d906..cea3ae405 100755
--- a/src/arch/isa_parser/operand_list.py
+++ b/src/arch/isa_parser/operand_list.py
@@ -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