qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dynge


From: Max Filippov
Subject: Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h
Date: Thu, 26 Apr 2012 00:56:10 +0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120329 Thunderbird/11.0.1

On 04/26/2012 12:31 AM, Peter Portante wrote:
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<address@hidden>
---
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?

Yes, you're right and the change itself is fine, but the patch is based on the 
old version of xtensa-semi.c,
that had #include "helpers.h". It was renamed to "helper.h" in the commit 
16c1deae215d4aac5b9b4fc090844b92852a0c5b.

--
Thanks.
-- Max



reply via email to

[Prev in Thread] Current Thread [Next in Thread]