Discussion:
[gem5-dev] Change in gem5/gem5[master]: systemc: set endianess to fix build for ARM host
Ciro Santilli (Gerrit)
2018-11-23 17:55:47 UTC
Permalink
Ciro Santilli has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/14595


Change subject: systemc: set endianess to fix build for ARM host
......................................................................

systemc: set endianess to fix build for ARM host

ARM architectures were not covered in the endianess #if cases, which
lead the build to fail on an arm host with message:

error The file boost/detail/endian.hpp needs to be set up for your CPU
type.

Change-Id: Id012cf37810da113174a51746e290e25138739cb
---
M src/systemc/ext/utils/endian.hh
1 file changed, 2 insertions(+), 1 deletion(-)



diff --git a/src/systemc/ext/utils/endian.hh
b/src/systemc/ext/utils/endian.hh
index fcf47e8..86e05cc 100644
--- a/src/systemc/ext/utils/endian.hh
+++ b/src/systemc/ext/utils/endian.hh
@@ -67,7 +67,8 @@
|| defined(_M_ALPHA) || defined(__amd64) \
|| defined(__amd64__) || defined(_M_AMD64) \
|| defined(__x86_64) || defined(__x86_64__) \
- || defined(_M_X64) || defined(__bfin__)
+ || defined(_M_X64) || defined(__bfin__) \
+ || defined(__arm__) || defined(__aarch64__)

# define SC_BOOST_LITTLE_ENDIAN
# define SC_BOOST_BYTE_ORDER 1234
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14595
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Id012cf37810da113174a51746e290e25138739cb
Gerrit-Change-Number: 14595
Gerrit-PatchSet: 1
Gerrit-Owner: Ciro Santilli <***@arm.com>
Gerrit-MessageType: newchange
Ciro Santilli (Gerrit)
2018-11-27 11:46:24 UTC
Permalink
Ciro Santilli has submitted this change and it was merged. (
https://gem5-review.googlesource.com/c/public/gem5/+/14595 )

Change subject: systemc: set endianess to fix build for ARM host
......................................................................

systemc: set endianess to fix build for ARM host

ARM architectures were not covered in the endianess #if cases, which
lead the build to fail on an arm host with message:

error The file boost/detail/endian.hpp needs to be set up for your CPU
type.

Change-Id: Id012cf37810da113174a51746e290e25138739cb
Reviewed-on: https://gem5-review.googlesource.com/c/14595
Reviewed-by: Gabe Black <***@google.com>
Maintainer: Gabe Black <***@google.com>
---
M src/systemc/ext/utils/endian.hh
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
Gabe Black: Looks good to me, approved; Looks good to me, approved



diff --git a/src/systemc/ext/utils/endian.hh
b/src/systemc/ext/utils/endian.hh
index fcf47e8..86e05cc 100644
--- a/src/systemc/ext/utils/endian.hh
+++ b/src/systemc/ext/utils/endian.hh
@@ -67,7 +67,8 @@
|| defined(_M_ALPHA) || defined(__amd64) \
|| defined(__amd64__) || defined(_M_AMD64) \
|| defined(__x86_64) || defined(__x86_64__) \
- || defined(_M_X64) || defined(__bfin__)
+ || defined(_M_X64) || defined(__bfin__) \
+ || defined(__arm__) || defined(__aarch64__)

# define SC_BOOST_LITTLE_ENDIAN
# define SC_BOOST_BYTE_ORDER 1234
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14595
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Id012cf37810da113174a51746e290e25138739cb
Gerrit-Change-Number: 14595
Gerrit-PatchSet: 2
Gerrit-Owner: Ciro Santilli <***@arm.com>
Gerrit-Reviewer: Ciro Santilli <***@arm.com>
Gerrit-Reviewer: Gabe Black <***@google.com>
Gerrit-Reviewer: Giacomo Travaglini <***@arm.com>
Gerrit-MessageType: merged
Loading...