arch: Wrap InstObjParams with a class and not a function.
authorGabe Black <gabe.black@gmail.com>
Sun, 17 Jan 2021 09:28:07 +0000 (01:28 -0800)
committerGabe Black <gabe.black@gmail.com>
Tue, 19 Jan 2021 00:12:19 +0000 (00:12 +0000)
commit544a172800e74b706d32024db83b535ef584daef
tree8811c2d823f316737c1b153cd7f3774693034f35
parentc2a980a58859d234171784fb54cbec3f6ad3762d
arch: Wrap InstObjParams with a class and not a function.

When parsing an ISA description, the InstObjParams class needs to have a
reference to the current parser. It does that by exposing a wrapper to
the description rather than the actual InstObjParams class. That wrapper
injects an additional argument into the InstObjParams constructor.

Originally, the wrapper which injectect the additional argument was a
function which masqueraded as a class. That made it impossible to
subclass InstObjParams.

Instead, this change replaces that function wrapper with a class
wrapper, and injects the extra argument in the __init__ method. This
preserves the fact that the InstObjParams name refers to a class, and
allows any sort of interaction that's normally allowed with a class like
subclassing.

Change-Id: I550ea2e60eadac3c7c0b9afa7d71f4607b49a5d2
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/39275
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/isa_parser.py