This constant is, first, a #define, and second only used in one place. In that one place, it appears that the code it guards is no longer necessary in general. It was originally written to avoid refetching a block of data that you're still in, even if you've moved slightly farther in it because you're skipping the next instruction due to an annulled branch delay slot. In reality however, in SPARC, the one ISA I'm aware of which has this sort of branching behavior, the PC state object will correctly determine that no branch is happening in these cases. Code lower down in the loop will then recompute where fetching should continue based on the next PC, automatically skipping the annulled branch slot without misinterpretting the gap as a branch. This change therefore also removes this block of code. Change-Id: I820ebc9df10aeb4fcb69c12f6a784e9ec616743c Reviewed-on: https://gem5-review.googlesource.com/6821 Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com> Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
72 lines
2.6 KiB
C++
72 lines
2.6 KiB
C++
/*
|
|
* Copyright (c) 2003-2005 The Regents of The University of Michigan
|
|
* Copyright (c) 2007-2008 The Florida State University
|
|
* Copyright (c) 2009 The University of Edinburgh
|
|
* 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.
|
|
*
|
|
* Authors: Timothy M. Jones
|
|
* Gabe Black
|
|
* Stephen Hines
|
|
*/
|
|
|
|
#ifndef __ARCH_POWER_ISA_TRAITS_HH__
|
|
#define __ARCH_POWER_ISA_TRAITS_HH__
|
|
|
|
#include "arch/power/types.hh"
|
|
#include "base/types.hh"
|
|
#include "cpu/static_inst_fwd.hh"
|
|
|
|
namespace BigEndianGuest {}
|
|
|
|
namespace PowerISA
|
|
{
|
|
|
|
using namespace BigEndianGuest;
|
|
|
|
StaticInstPtr decodeInst(ExtMachInst);
|
|
|
|
const Addr PageShift = 12;
|
|
const Addr PageBytes = ULL(1) << PageShift;
|
|
const Addr Page_Mask = ~(PageBytes - 1);
|
|
const Addr PageOffset = PageBytes - 1;
|
|
|
|
const Addr PteShift = 3;
|
|
const Addr NPtePageShift = PageShift - PteShift;
|
|
const Addr NPtePage = ULL(1) << NPtePageShift;
|
|
const Addr PteMask = NPtePage - 1;
|
|
|
|
const int MachineBytes = 4;
|
|
|
|
// Memory accesses can be unaligned
|
|
const bool HasUnalignedMemAcc = true;
|
|
|
|
const bool CurThreadInfoImplemented = false;
|
|
const int CurThreadInfoReg = -1;
|
|
|
|
} // namespace PowerISA
|
|
|
|
#endif // __ARCH_POWER_ISA_TRAITS_HH__
|