In this context, the decoder width is the number of bytes that are fed into the decoder at once. This is frequently the same as the size of an instruction, but in instructions with occasionally variable instruction sizes (ARM, RISCV), or extremely variable instruction sizes (x86) there may be no relation. Rather than determining the amount of data to feed to the decoder based on a MachInst type defined by each ISA, this new interface adds some new properties to the base InstDecoder class each arch specific decoder inherits from. These are the size of the incoming buffer, a pointer to wherever that data should end up, and a mask for masking a PC value so it aligns with the instruction size. These values are filled in by a templated InstDecoder constructor which is templated based on what would have historically been the MachInst type. Because the "moreBytes" method would historically accept a parameter of type MachInst, this parameter has also been eliminated. Now, the decoder's parent object should use the pointer and size values to fill in the buffer moreBytes reads. Then when moreBytes is called, it just uses the buffer without having to show what its type is externally. Change-Id: I0642cdb6a61e152441ca4ce47d748639175cda90 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40175 Reviewed-by: Gabe Black <gabe.black@gmail.com> Maintainer: Gabe Black <gabe.black@gmail.com> Tested-by: kokoro <noreply+kokoro@google.com>
119 lines
3.4 KiB
C++
119 lines
3.4 KiB
C++
/*
|
|
* Copyright (c) 2012 Google
|
|
* All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions are
|
|
* met: redistributions of source code must retain the above copyright
|
|
* notice, this list of conditions and the following disclaimer;
|
|
* redistributions in binary form must reproduce the above copyright
|
|
* notice, this list of conditions and the following disclaimer in the
|
|
* documentation and/or other materials provided with the distribution;
|
|
* neither the name of the copyright holders nor the names of its
|
|
* contributors may be used to endorse or promote products derived from
|
|
* this software without specific prior written permission.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
|
|
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
|
|
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
|
|
* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
|
|
* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
|
|
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
|
|
* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
|
|
* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
|
|
* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
|
|
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
|
|
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
|
|
*/
|
|
|
|
#ifndef __ARCH_POWER_DECODER_HH__
|
|
#define __ARCH_POWER_DECODER_HH__
|
|
|
|
#include "arch/generic/decode_cache.hh"
|
|
#include "arch/generic/decoder.hh"
|
|
#include "arch/power/types.hh"
|
|
#include "cpu/static_inst.hh"
|
|
#include "debug/Decode.hh"
|
|
|
|
namespace PowerISA
|
|
{
|
|
|
|
class ISA;
|
|
class Decoder : public InstDecoder
|
|
{
|
|
protected:
|
|
// The extended machine instruction being generated
|
|
ExtMachInst emi;
|
|
bool instDone;
|
|
|
|
public:
|
|
Decoder(ISA* isa=nullptr) : InstDecoder(&emi), instDone(false) {}
|
|
|
|
void
|
|
process()
|
|
{
|
|
}
|
|
|
|
void
|
|
reset()
|
|
{
|
|
instDone = false;
|
|
}
|
|
|
|
// Use this to give data to the predecoder. This should be used
|
|
// when there is control flow.
|
|
void
|
|
moreBytes(const PCState &pc, Addr fetchPC)
|
|
{
|
|
emi = betoh(emi);
|
|
instDone = true;
|
|
}
|
|
|
|
bool
|
|
needMoreBytes()
|
|
{
|
|
return true;
|
|
}
|
|
|
|
bool
|
|
instReady()
|
|
{
|
|
return instDone;
|
|
}
|
|
|
|
void takeOverFrom(Decoder *old) {}
|
|
|
|
protected:
|
|
/// A cache of decoded instruction objects.
|
|
static GenericISA::BasicDecodeCache<Decoder, ExtMachInst> defaultCache;
|
|
friend class GenericISA::BasicDecodeCache<Decoder, ExtMachInst>;
|
|
|
|
StaticInstPtr decodeInst(ExtMachInst mach_inst);
|
|
|
|
/// Decode a machine instruction.
|
|
/// @param mach_inst The binary instruction to decode.
|
|
/// @retval A pointer to the corresponding StaticInst object.
|
|
StaticInstPtr
|
|
decode(ExtMachInst mach_inst, Addr addr)
|
|
{
|
|
StaticInstPtr si = defaultCache.decode(this, mach_inst, addr);
|
|
DPRINTF(Decode, "Decode: Decoded %s instruction: %#x\n",
|
|
si->getName(), mach_inst);
|
|
return si;
|
|
}
|
|
|
|
public:
|
|
StaticInstPtr
|
|
decode(PowerISA::PCState &nextPC)
|
|
{
|
|
if (!instDone)
|
|
return NULL;
|
|
instDone = false;
|
|
return decode(emi, nextPC.instAddr());
|
|
}
|
|
};
|
|
|
|
} // namespace PowerISA
|
|
|
|
#endif // __ARCH_POWER_DECODER_HH__
|