summaryrefslogtreecommitdiff
path: root/src/sim
diff options
context:
space:
mode:
authorGabe Black <gabeblack@google.com>2019-10-29 18:56:27 -0700
committerGabe Black <gabeblack@google.com>2019-11-18 20:02:31 +0000
commitd40f0bc579fb8b10da7181d3a144cd3e9a0a0e59 (patch)
treeeb0a80fd0a21017befa4eee8c4e77380bb1b4657 /src/sim
parent697e55995626f24658ce443287cd1ba90c2f68eb (diff)
downloadgem5-d40f0bc579fb8b10da7181d3a144cd3e9a0a0e59.tar.xz
arch: Get rid of the (Big|Little)EndianGuest namespaces.
These namespaces were used to set up an environment/context where there was an implicit guest namespace. This is an issue when there may be multiple guest endiannesses which might be different. In cases where we don't know what the guest endianness is, we can't rely on it being an implicit part of our context since that would be ambiguous. In cases where we do know, for instance in ISA specific code, we can just use the endianness specific version that's appropriate for that context. This also (somewhat) removes the assumption that there is a single endianness that applies for a particular ISA. Practically speaking this assumption will probably still stand though, since there would likely be a non-trivial performance penalty to apply a configurable endianness instead of a fixed one the compiler can optomize/remove. Change-Id: I2dff338b58726d724f387388efe32d9233885680 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/22374 Reviewed-by: Jason Lowe-Power <jason@lowepower.com> Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com> Maintainer: Gabe Black <gabeblack@google.com> Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'src/sim')
-rw-r--r--src/sim/byteswap.hh33
1 files changed, 0 insertions, 33 deletions
diff --git a/src/sim/byteswap.hh b/src/sim/byteswap.hh
index 0b41c6779..6745e0060 100644
--- a/src/sim/byteswap.hh
+++ b/src/sim/byteswap.hh
@@ -169,37 +169,4 @@ inline T gtoh(T value, ByteOrder guest_byte_order)
betoh(value) : letoh(value);
}
-namespace BigEndianGuest
-{
- const ByteOrder GuestByteOrder = BigEndianByteOrder;
- template <typename T>
- inline T gtole(T value) {return betole(value);}
- template <typename T>
- inline T letog(T value) {return letobe(value);}
- template <typename T>
- inline T gtobe(T value) {return value;}
- template <typename T>
- inline T betog(T value) {return value;}
- template <typename T>
- inline T htog(T value) {return htobe(value);}
- template <typename T>
- inline T gtoh(T value) {return betoh(value);}
-}
-
-namespace LittleEndianGuest
-{
- const ByteOrder GuestByteOrder = LittleEndianByteOrder;
- template <typename T>
- inline T gtole(T value) {return value;}
- template <typename T>
- inline T letog(T value) {return value;}
- template <typename T>
- inline T gtobe(T value) {return letobe(value);}
- template <typename T>
- inline T betog(T value) {return betole(value);}
- template <typename T>
- inline T htog(T value) {return htole(value);}
- template <typename T>
- inline T gtoh(T value) {return letoh(value);}
-}
#endif // __SIM_BYTE_SWAP_HH__