arch-arm,cpu: Introduce a getEMI virtual method on StaticInst.
authorGabe Black <gabe.black@gmail.com>
Fri, 29 Jan 2021 01:18:26 +0000 (17:18 -0800)
committerGabe Black <gabe.black@gmail.com>
Wed, 3 Feb 2021 06:08:37 +0000 (06:08 +0000)
commite656730f356edf86879abc522862c267c90ab3de
tree2f0bb63a0a45a3d8486e16dc78f798341bca93dd
parenteb7acc7c6567bbc47b3adaf6a5d84c25afa85bba
arch-arm,cpu: Introduce a getEMI virtual method on StaticInst.

This takes the place of direct access to the machInst field as used in
the MinorCPU model which makes the incorrect assumption that it can
arbitrarily treat the ExtMachInst as an integer, and that masking in a
certain way can meaningfully classify what the instruction will do.

Because that assumption is not correct in general, that had been
ifdef-ed out in most ISAs except ARM, and for the other ISAs the value
was simply set to zero.

Change-Id: I8ac05e65475edc3ccc044afdff09490e2c05ba07
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40098
Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Maintainer: Giacomo Travaglini <giacomo.travaglini@arm.com>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/arm/insts/static_inst.hh
src/cpu/minor/func_unit.cc
src/cpu/static_inst.hh