null: Stop specifying an endianness in isa_traits.hh.
authorGabe Black <gabeblack@google.com>
Fri, 12 Oct 2018 12:15:34 +0000 (05:15 -0700)
committerGabe Black <gabeblack@google.com>
Thu, 18 Oct 2018 07:48:10 +0000 (07:48 +0000)
The NULL ISA doesn't really have an endianness. Now that the packet
accessors which consumed that endianness are gone, we can get rid of
that setting as well.

Change-Id: I8dd4c7b8236b07df4458fea377865f30141121d4
Reviewed-on: https://gem5-review.googlesource.com/c/13466
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/arch/null/isa_traits.hh
src/mem/packet.hh
src/mem/packet_access.hh

index 8c45dd06bbedd1fb70c8156dcb8982c3849094e4..d15f83cb43db3612598fa7a7eb66b94963b24629 100644 (file)
 
 #include "base/types.hh"
 
-namespace LittleEndianGuest {}
-
 namespace NullISA
 {
-    using namespace LittleEndianGuest;
-
     const Addr PageShift = 12;
     const Addr PageBytes = ULL(1) << PageShift;
-
 }
 
 #endif //__ARCH_NULL_ISA_TRAITS_HH__
index d82ca4f3e07a26dae46dfb9df637132fba9d8754..95c6f81834a888152c680130cad6a61d92e38110 100644 (file)
@@ -63,6 +63,7 @@
 #include "base/logging.hh"
 #include "base/printable.hh"
 #include "base/types.hh"
+#include "config/the_isa.hh"
 #include "mem/request.hh"
 #include "sim/core.hh"
 
@@ -1055,12 +1056,14 @@ class Packet : public Printable
     template <typename T>
     T get(ByteOrder endian) const;
 
+#if THE_ISA != NULL_ISA
     /**
      * Get the data in the packet byte swapped from guest to host
      * endian.
      */
     template <typename T>
     T get() const;
+#endif
 
     /** Set the value in the data pointer to v as big endian. */
     template <typename T>
@@ -1077,9 +1080,11 @@ class Packet : public Printable
     template <typename T>
     void set(T v, ByteOrder endian);
 
+#if THE_ISA != NULL_ISA
     /** Set the value in the data pointer to v as guest endian. */
     template <typename T>
     void set(T v);
+#endif
 
 
     /**
index 92752a72011096631c0617421abee56a0b2aa8a9..97357ae5e355406902c1d5fc0af1658f2ebe4185 100644 (file)
@@ -43,7 +43,6 @@
  */
 
 #include "arch/isa_traits.hh"
-#include "config/the_isa.hh"
 #include "mem/packet.hh"
 #include "sim/byteswap.hh"
 
@@ -99,12 +98,14 @@ Packet::get(ByteOrder endian) const
     };
 }
 
+#if THE_ISA != NULL_ISA
 template <typename T>
 inline T
 Packet::get() const
 {
     return TheISA::gtoh(getRaw<T>());
 }
+#endif
 
 template <typename T>
 inline void
@@ -136,11 +137,13 @@ Packet::set(T v, ByteOrder endian)
     };
 }
 
+#if THE_ISA != NULL_ISA
 template <typename T>
 inline void
 Packet::set(T v)
 {
     setRaw(TheISA::htog(v));
 }
+#endif
 
 #endif //__MEM_PACKET_ACCESS_HH__