sim: mips: fix missing prototype in multi-run generation
authorMike Frysinger <vapier@gentoo.org>
Thu, 4 Nov 2021 03:44:19 +0000 (23:44 -0400)
committerMike Frysinger <vapier@gentoo.org>
Thu, 4 Nov 2021 03:53:10 +0000 (23:53 -0400)
The multi-run logic for mips involves a bit of codegen and rewriting
of files to include per-architecture prefixes.  That can result in
files with missing prototypes which cause compiler errors.  In the
case of mips-sde-elf targets, we have:
$srcdir/m16run.c -> $builddir/m16mips64r2_run.c
  sim_engine_run -> m16mips64r2_engine_run
$srcdir/micromipsrun.c -> micromipsmicromips_run.c
  sim_engine_run -> micromips64micromips_engine_run

micromipsmicromips_run.c:80:1: error: no previous prototype for 'micromips64micromips_engine_run' [-Werror=missing-prototypes]
   80 | micromips64micromips_engine_run (SIM_DESC sd, int next_cpu_nr, int nr_cpus,

We generate headers for those prototypes in the configure script,
but only include them in the generated multi-run.c file.  Update the
rewrite logic to turn the sim-engine.h include into the relevant
generated engine include so these files also have their prototypes.
$srcdir/m16run.c -> $builddir/m16mips64r2_run.c
  sim-engine.h -> m16mips64r2_engine.h
$srcdir/micromipsrun.c -> micromipsmicromips_run.c
  sim-engine.h -> micromips64micromips_engine.h

sim/mips/Makefile.in
sim/mips/m16run.c

index b34fdf5b461e54043610e2c3b7ef78ca71a47eb0..589c392edb5338082e82e7464d56d61ef3adfeee 100644 (file)
@@ -553,6 +553,7 @@ tmp-run-multi: $(srcdir)/m16run.c $(srcdir)/micromipsrun.c
              m=`echo $${t} | sed -e 's/^m16//' -e 's/:.*//'`; \
              sed <  $(srcdir)/m16run.c > tmp-run \
                    -e "s/^sim_/m16$${m}_/" \
+                   -e "/include/s/sim-engine/m16$${m}_engine/" \
                    -e "s/m16_/m16$${m}_/" \
                    -e "s/m32_/m32$${m}_/" ; \
              $(SHELL) $(srcdir)/../../move-if-change tmp-run \
@@ -562,6 +563,7 @@ tmp-run-multi: $(srcdir)/m16run.c $(srcdir)/micromipsrun.c
              m=`echo $${t} | sed -e 's/^micromips32//' -e 's/:.*//'`; \
              sed <  $(srcdir)/micromipsrun.c > tmp-run \
                    -e "s/^sim_/micromips32$${m}_/" \
+                   -e "/include/s/sim-engine/micromips32$${m}_engine/" \
                    -e "s/micromips16_/micromips16$${m}_/" \
                    -e "s/micromips32_/micromips32$${m}_/" \
                    -e "s/m32_/m32$${m}_/" ; \
@@ -572,6 +574,7 @@ tmp-run-multi: $(srcdir)/m16run.c $(srcdir)/micromipsrun.c
              m=`echo $${t} | sed -e 's/^micromips64//' -e 's/:.*//'`; \
              sed <  $(srcdir)/micromipsrun.c > tmp-run \
                    -e "s/^sim_/micromips64$${m}_/" \
+                   -e "/include/s/sim-engine/micromips64$${m}_engine/" \
                    -e "s/micromips16_/micromips16$${m}_/" \
                    -e "s/micromips32_/micromips64$${m}_/" \
                    -e "s/m32_/m64$${m}_/" ; \
index a04e3eed3bf3868b4a24d8feec66020d9c611d53..bbe487e4e513f0cd0b841dc72e899d8a5513c0d2 100644 (file)
@@ -24,6 +24,7 @@
 #include "m16_idecode.h"
 #include "m32_idecode.h"
 #include "bfd.h"
+#include "sim-engine.h"
 
 
 #define SD sd