Re: [PATCH v2 22/26] target/arm/tcg: Reduce 'helper-m.h.inc' inclusion

2023-06-14 Thread Philippe Mathieu-Daudé
On 14/6/23 06:55, Richard Henderson wrote: On 6/11/23 10:58, Philippe Mathieu-Daudé wrote: +++ b/target/arm/tcg/translate.c @@ -37,6 +37,11 @@   #include "exec/helper-gen.h.inc"   #undef  HELPER_H +#define HELPER_H "tcg/helper-m.h.inc" +#include "exec/helper-proto.h.inc" +#include "exec/helper-g

Re: [PATCH v2 22/26] target/arm/tcg: Reduce 'helper-m.h.inc' inclusion

2023-06-13 Thread Richard Henderson
On 6/11/23 10:58, Philippe Mathieu-Daudé wrote: +++ b/target/arm/tcg/translate.c @@ -37,6 +37,11 @@ #include "exec/helper-gen.h.inc" #undef HELPER_H +#define HELPER_H "tcg/helper-m.h.inc" +#include "exec/helper-proto.h.inc" +#include "exec/helper-gen.h.inc" +#undef HELPER_H You don't

[PATCH v2 22/26] target/arm/tcg: Reduce 'helper-m.h.inc' inclusion

2023-06-11 Thread Philippe Mathieu-Daudé
Instead of including helper-m.h.inc via helper.h which is included by all TCG files, restrict it to the few files that require it. Signed-off-by: Philippe Mathieu-Daudé --- target/arm/helper.h | 2 -- target/arm/tcg/m_helper.c | 4 target/arm/tcg/translate-m-nocp.c | 6