diff options
author | Rekai Gonzalez Alberquilla <Rekai.GonzalezAlberquilla@arm.com> | 2015-10-09 14:50:54 -0500 |
---|---|---|
committer | Rekai Gonzalez Alberquilla <Rekai.GonzalezAlberquilla@arm.com> | 2015-10-09 14:50:54 -0500 |
commit | d3d159749a0a6c3b69a9181fab8db34b6ba0f7a1 (patch) | |
tree | 477a11d57b4428685247f1ad4bb5863b948f9c99 /src/arch/arm/decoder.cc | |
parent | 7624fc1fb461f1dd127763521d85f63e81617d71 (diff) | |
download | gem5-d3d159749a0a6c3b69a9181fab8db34b6ba0f7a1.tar.xz |
isa: Add parameter to pick different decoder inside ISA
The decoder is responsible for splitting instructions in micro
operations (uops). Given that different micro architectures may split
operations differently, this patch allows to specify which micro
architecture each isa implements, so different cores in the system can
split instructions differently, also decoupling uop splitting
(microArch) from ISA (Arch). This is done making the decodification
calls templates that receive a type 'DecoderFlavour' that maps the
name of the operation to the class that implements it. This way there
is only one selection point (converting the command line enum to the
appropriate DecodeFeatures object). In addition, there is no explicit
code replication: template instantiation hides that, and the compiler
should be able to resolve a number of things at compile-time.
Diffstat (limited to 'src/arch/arm/decoder.cc')
-rw-r--r-- | src/arch/arm/decoder.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/arch/arm/decoder.cc b/src/arch/arm/decoder.cc index 23fa89a3f..1502b061f 100644 --- a/src/arch/arm/decoder.cc +++ b/src/arch/arm/decoder.cc @@ -41,6 +41,8 @@ */ #include "arch/arm/decoder.hh" + +#include "arch/arm/isa.hh" #include "arch/arm/isa_traits.hh" #include "arch/arm/utility.hh" #include "base/trace.hh" @@ -51,8 +53,10 @@ namespace ArmISA GenericISA::BasicDecodeCache Decoder::defaultCache; -Decoder::Decoder() - : data(0), fpscrLen(0), fpscrStride(0) +Decoder::Decoder(ISA* isa) + : data(0), fpscrLen(0), fpscrStride(0), decoderFlavour(isa + ? isa->decoderFlavour() + : Enums::Generic) { reset(); } |