cpu: Removed unnecessary file reg_class_impl.hh

Previously, reg_class_impl.hh was added in order to prevent a cyclic
dependency between it and the_isa.hh (See
http://reviews.gem5.org/r/3754). It was determined that this was not
necessary. The two files had almost entirely the same includes, and the
current test-suite including multiple gcc and clang compilers on both
MacOS and Linux successfully built the library with all functionality
moved into the reg_class.hh file.

Change-Id: I0319e187b9eb280726a003951bb1ce315ffe17f5
Signed-off-by: Bradley Wang <radwang@ucdavis.edu>
Reviewed-on: https://gem5-review.googlesource.com/11869
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

diff --git a/src/cpu/o3/rename_map.cc b/src/cpu/o3/rename_map.cc
index bde2a692195de02b2ddc39ddd0d8bc33e58dde8d..1194b550ba262089179dde47a7bdba656ed5673f 100644 (file)
--- a/src/cpu/o3/rename_map.cc
+++ b/src/cpu/o3/rename_map.cc
@@ -45,7 +45,7 @@
 
 #include <vector>
 
-#include "cpu/reg_class_impl.hh"
+#include "cpu/reg_class.hh"
 #include "debug/Rename.hh"
 
 using namespace std;
@@ -193,4 +193,3 @@ UnifiedRenameMap::switchMode(VecMode newVecMode, UnifiedFreeList* freeList)
         vecMode = Enums::Full;
     }
 }
-

diff --git a/src/cpu/reg_class.hh b/src/cpu/reg_class.hh
index 27bf59b196fe4e2a4613cf939ca663034e2f12d6..def275ee446f7b9f3221db0bb8227fa15bd03eb2 100644 (file)
--- a/src/cpu/reg_class.hh
+++ b/src/cpu/reg_class.hh
@@ -131,7 +131,12 @@ class RegId {
      * constant zero value throughout the execution).
      */
 
-    inline bool isZeroReg() const;
+    inline bool isZeroReg() const
+    {
+        return ((regClass == IntRegClass && regIdx == TheISA::ZeroReg) ||
+               (THE_ISA == ALPHA_ISA && regClass == FloatRegClass &&
+                regIdx == TheISA::ZeroReg));
+    }
 
     /** @return true if it is an integer physical register. */
     bool isIntReg() const { return regClass == IntRegClass; }
@@ -167,7 +172,21 @@ class RegId {
     /** Index flattening.
      * Required to be able to use a vector for the register mapping.
      */
-    inline RegIndex flatIndex() const;
+    inline RegIndex flatIndex() const
+    {
+        switch (regClass) {
+          case IntRegClass:
+          case FloatRegClass:
+          case VecRegClass:
+          case CCRegClass:
+          case MiscRegClass:
+            return regIdx;
+          case VecElemClass:
+            return Scale*regIdx + elemIdx;
+        }
+        panic("Trying to flatten a register without class!");
+        return -1;
+    }
     /** @} */
 
     /** Elem accessor */

diff --git a/src/cpu/reg_class_impl.hh b/src/cpu/reg_class_impl.hh
deleted file mode 100644 (file)
index 98b341e..0000000
--- a/src/cpu/reg_class_impl.hh
+++ /dev/null
@@ -1,75 +0,0 @@
-/*
- * Copyright (c) 2016 ARM Limited
- * All rights reserved
- *
- * The license below extends only to copyright in the software and shall
- * not be construed as granting a license to any other intellectual
- * property including but not limited to intellectual property relating
- * to a hardware implementation of the functionality of the software
- * licensed hereunder.  You may use the software subject to the license
- * terms below provided that you ensure that this notice is replicated
- * unmodified and in its entirety in all distributions of the software,
- * modified or unmodified, in source code or in binary form.
- *
- * 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: Rekai Gonzalez
- */
-
-#ifndef __CPU__REG_CLASS_IMPL_HH__
-#define __CPU__REG_CLASS_IMPL_HH__
-
-#include <cassert>
-#include <cstddef>
-#include <iostream>
-
-#include "arch/registers.hh"
-#include "config/the_isa.hh"
-#include "cpu/reg_class.hh"
-
-bool RegId::isZeroReg() const
-{
-    return ((regClass == IntRegClass && regIdx == TheISA::ZeroReg) ||
-            (THE_ISA == ALPHA_ISA && regClass == FloatRegClass &&
-             regIdx == TheISA::ZeroReg));
-}
-
-static constexpr size_t Scale = TheISA::NumVecElemPerVecReg;
-
-RegIndex RegId::flatIndex() const {
-    switch (regClass) {
-    case IntRegClass:
-    case FloatRegClass:
-    case VecRegClass:
-    case CCRegClass:
-    case MiscRegClass:
-        return regIdx;
-    case VecElemClass:
-        return Scale*regIdx + elemIdx;
-    }
-    panic("Trying to flatten a register without class!");
-    return -1;
-}
-
-#endif // __CPU__REG_CLASS_IMPL_HH__

diff --git a/src/cpu/static_inst.hh b/src/cpu/static_inst.hh
index 16b5ffd3c2d4343ae7846dc1dce41328609ed616..70d933c315ac761dc0ec8eeb4a6da63739e827f0 100644 (file)
--- a/src/cpu/static_inst.hh
+++ b/src/cpu/static_inst.hh
@@ -44,7 +44,6 @@
 #include "config/the_isa.hh"
 #include "cpu/op_class.hh"
 #include "cpu/reg_class.hh"
-#include "cpu/reg_class_impl.hh"
 #include "cpu/static_inst_fwd.hh"
 #include "cpu/thread_context.hh"
 #include "enums/StaticInstFlags.hh"