On 30/04/2021 00:40, Richard Henderson wrote:
On 4/29/21 9:21 AM, Bruno Larsen (billionai) wrote:
@@ -234,19 +235,19 @@ static void spr_read_tbu(DisasContext *ctx, int
gprn, int sprn)
}
ATTRIBUTE_UNUSED
-static void spr_read_atbl(DisasContext *ctx, int gprn, int sprn)
+void spr_read_atbl(DisasContext *ctx, int gprn, int sprn)
{
You can drop the ATTRIBUTE_UNUSED at the same time.
It was only there to stop the static symbol from being signaled as
unused; for a non-static symbol, the compiler obviously can't tell.
ah, that makes sense. Dropped it, thanks!
diff --git a/target/ppc/spr_tcg.h b/target/ppc/spr_tcg.h
new file mode 100644
index 0000000000..b573a23e7b
--- /dev/null
+++ b/target/ppc/spr_tcg.h
@@ -0,0 +1,121 @@
+#ifndef SPR_TCG_H
+#define SPR_TCG_H
+
+#include "qemu/osdep.h"
+#include "cpu.h"
+#include "exec/translator.h"
+#include "tcg/tcg.h"
All new files get copyright headers. No headers include osdep.h. It
doesn't appear that you need any headers for this file; just add
typedef struct DisasContext DisasContext;
to the top of the file.
Apparently, I don't even need to do that. I'm not sure what I've
changed, but now I don't need neither the cpu.h nor the typedef. I can
just not include anything, so I dropped them all.
r~
--
Bruno Piazera Larsen Instituto de Pesquisas ELDORADO
<https://www.eldorado.org.br/?utm_campaign=assinatura_de_e-mail&utm_medium=email&utm_source=RD+Station>
Departamento Computação Embarcada Analista de Software Trainee Aviso
Legal - Disclaimer <https://www.eldorado.org.br/disclaimer.html>
