On 04/25/2012 04:07 PM, Max Filippov wrote:
On 04/25/2012 10:47 PM, Peter Portante wrote:
Signed-off-by: Peter Portante<peter.porta...@redhat.com>
---
hw/spapr_hcall.c | 1 -
xtensa-semi.c | 1 -
2 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c
index 94bb504..88c1fab 100644
--- a/hw/spapr_hcall.c
+++ b/hw/spapr_hcall.c
@@ -1,6 +1,5 @@
#include "sysemu.h"
#include "cpu.h"
-#include "dyngen-exec.h"
#include "qemu-char.h"
#include "sysemu.h"
#include "qemu-char.h"
diff --git a/xtensa-semi.c b/xtensa-semi.c
index 5754b77..8c97a02 100644
--- a/xtensa-semi.c
+++ b/xtensa-semi.c
@@ -30,7 +30,6 @@
#include<string.h>
#include<stddef.h>
#include "cpu.h"
-#include "dyngen-exec.h"
#include "helpers.h"
#include "qemu-log.h"
Regarding xtensa part: it does not apply now that helpers.h is renamed
to helper.h
Hi Max,
I am not entirely sure I understand what you wrote above. I am probably
missing a subtlety above, so pardon me ahead of time.
For the file xtensa-semi.c, the references it makes to "env" are
satisfied by the argument declaration. Looking through helper.h and
def-helper.h, for completeness, I don't see any references to "env"
either. So it appears that dyngen-exec.h does not need to be included
(and in fact, it is born out by a successful compile).
Do I understand things correctly?
Thanks, -peter
Otherwise Acked-by: Max Filippov <jcmvb...@gmail.com>
