diff mbox

sim: move sim-io.h to sim-assert.h

Message ID 1426319106-19298-1-git-send-email-vapier@gentoo.org
State Committed
Headers

Commit Message

Mike Frysinger March 14, 2015, 7:45 a.m. UTC 
  Since sim-endian.c doesn't actually use sim_io funcs, it's weird to
include the sim-io.h header here.  It's doing so only for the assert
header.  So lets relocate the include to the right place.

Committed.
---
 sim/common/ChangeLog    | 6 ++++++
 sim/common/sim-assert.h | 2 ++
 sim/common/sim-endian.c | 1 -
 3 files changed, 8 insertions(+), 1 deletion(-)
diff mbox

Patch

diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index 265758f..95ae4a4 100644
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,9 @@ 
+2015-03-14  Mike Frysinger  <vapier@gentoo.org>
+
+	* sim-assert.h [!SIM_ASSERT, WITH_ASSERT]: Include sim-io.h.
+	[!ASSERT, WITH_ASSERT]: Likewise.
+	* sim-endian.c: Delete sim-io.h include.
+
 2015-02-19  Mike Frysinger  <vapier@gentoo.org>
 
 	* run-sim.h (sim_kill): Delete unused prototype.
diff --git a/sim/common/sim-assert.h b/sim/common/sim-assert.h
index 3de8114..eaaf791 100644
--- a/sim/common/sim-assert.h
+++ b/sim/common/sim-assert.h
@@ -42,6 +42,7 @@  while (0)
 
 #if !defined (SIM_ASSERT)
 #if defined (WITH_ASSERT)
+#include "sim-io.h"
 #define SIM_ASSERT(EXPRESSION) \
 do \
   { \
@@ -65,6 +66,7 @@  while (0)
 
 #if !defined (ASSERT)
 #if defined (WITH_ASSERT)
+#include "sim-io.h"
 #define ASSERT(EXPRESSION) \
 do \
   { \
diff --git a/sim/common/sim-endian.c b/sim/common/sim-endian.c
index d2e19b7..9e256c3 100644
--- a/sim/common/sim-endian.c
+++ b/sim/common/sim-endian.c
@@ -25,7 +25,6 @@ 
 
 #include "sim-basics.h"
 #include "sim-assert.h"
-#include "sim-io.h"
 
 
 #if !defined(_SWAP_1)