213c9186de
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>
59 lines
2.3 KiB
C++
59 lines
2.3 KiB
C++
/*
|
|
* Copyright 2020 Google, Inc.
|
|
*
|
|
* 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_GENERIC_DECODER_HH__
|
|
#define __ARCH_GENERIC_DECODER_HH__
|
|
|
|
#include "base/bitfield.hh"
|
|
#include "base/intmath.hh"
|
|
#include "base/types.hh"
|
|
#include "cpu/static_inst_fwd.hh"
|
|
|
|
class InstDecoder
|
|
{
|
|
protected:
|
|
void *_moreBytesPtr;
|
|
size_t _moreBytesSize;
|
|
Addr _pcMask;
|
|
|
|
public:
|
|
template <typename MoreBytesType>
|
|
InstDecoder(MoreBytesType *mb_buf) :
|
|
_moreBytesPtr(mb_buf), _moreBytesSize(sizeof(MoreBytesType)),
|
|
_pcMask(~mask(floorLog2(_moreBytesSize)))
|
|
{}
|
|
|
|
virtual StaticInstPtr fetchRomMicroop(
|
|
MicroPC micropc, StaticInstPtr curMacroop);
|
|
|
|
void *moreBytesPtr() const { return _moreBytesPtr; }
|
|
size_t moreBytesSize() const { return _moreBytesSize; }
|
|
Addr pcMask() const { return _pcMask; }
|
|
};
|
|
|
|
#endif // __ARCH_DECODER_GENERIC_HH__
|