[gem5-dev] Change in gem5/gem5[develop]: systemc: remove boost header dependency

2021-02-02 Thread Earl Ou (Gerrit) via gem5-dev
Earl Ou has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/40315 )


Change subject: systemc: remove boost header dependency
..

systemc: remove boost header dependency

The current tests included don't require boost header to work. Remove
the dependency. This also gets rid of the warning message generated by
the latest boost headers.

Tested by running systemC tests:

src/systemc/tests/verify.py --update-json \
  --filter-file=src/systemc/tests/working.filt -j 56 build/ARM/

Change-Id: I9d3bfe145597335abdf24f2de85ed3c0708aea27
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40315
Reviewed-by: Gabe Black 
Maintainer: Gabe Black 
Tested-by: kokoro 
---
M src/systemc/ext/systemc
1 file changed, 0 insertions(+), 6 deletions(-)

Approvals:
  Gabe Black: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/systemc/ext/systemc b/src/systemc/ext/systemc
index 60ec1e6..6968b4c 100644
--- a/src/systemc/ext/systemc
+++ b/src/systemc/ext/systemc
@@ -28,12 +28,6 @@
 #ifndef __SYSTEMC_EXT_SYSTEMC__
 #define __SYSTEMC_EXT_SYSTEMC__

-// This include isn't supposed to be necessary, but some regression tests
-// assume that the sc_bind macro will work without explicitly including the
-// boost headers. This is in contradiction to the spec which says boost  
isn't

-// a required dependency.
-#include 
-
 #include "channel/_channel.hh"
 #include "core/_core.hh"
 #include "dt/_dt.hh"

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/40315
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I9d3bfe145597335abdf24f2de85ed3c0708aea27
Gerrit-Change-Number: 40315
Gerrit-PatchSet: 3
Gerrit-Owner: Earl Ou 
Gerrit-Reviewer: Earl Ou 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: kokoro 
Gerrit-CC: Yu-hsin Wang 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: systemc: remove boost header dependency

2021-02-01 Thread Earl Ou (Gerrit) via gem5-dev
Earl Ou has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/40315 )



Change subject: systemc: remove boost header dependency
..

systemc: remove boost header dependency

The current tests included doesn't require boost header to work. Remove
the dependency. This also gets rid of the warning message generated by
the latest boost headers.

Change-Id: I9d3bfe145597335abdf24f2de85ed3c0708aea27
---
M src/systemc/ext/systemc
1 file changed, 0 insertions(+), 6 deletions(-)



diff --git a/src/systemc/ext/systemc b/src/systemc/ext/systemc
index 60ec1e6..6968b4c 100644
--- a/src/systemc/ext/systemc
+++ b/src/systemc/ext/systemc
@@ -28,12 +28,6 @@
 #ifndef __SYSTEMC_EXT_SYSTEMC__
 #define __SYSTEMC_EXT_SYSTEMC__

-// This include isn't supposed to be necessary, but some regression tests
-// assume that the sc_bind macro will work without explicitly including the
-// boost headers. This is in contradiction to the spec which says boost  
isn't

-// a required dependency.
-#include 
-
 #include "channel/_channel.hh"
 #include "core/_core.hh"
 #include "dt/_dt.hh"

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/40315
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I9d3bfe145597335abdf24f2de85ed3c0708aea27
Gerrit-Change-Number: 40315
Gerrit-PatchSet: 1
Gerrit-Owner: Earl Ou 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s