Re: [PATCH v3 09/29] plugins: Include trace/mem.h in api.c

2020-01-04 Thread Philippe Mathieu-Daudé
On 1/3/20 10:59 PM, Richard Henderson wrote: On 1/3/20 5:22 PM, Philippe Mathieu-Daudé wrote: Hi Richard, On 12/29/19 12:11 AM, Richard Henderson wrote: Code movement in an upcoming patch will show that this file was implicitly depending on trace/mem.h being included beforehand. Ah, it uses

Re: [PATCH v3 09/29] plugins: Include trace/mem.h in api.c

2020-01-03 Thread Richard Henderson
On 1/3/20 5:22 PM, Philippe Mathieu-Daudé wrote: > Hi Richard, > > On 12/29/19 12:11 AM, Richard Henderson wrote: >> Code movement in an upcoming patch will show that this file >> was implicitly depending on trace/mem.h being included beforehand. > > Ah, it uses the TRACE_MEM_* macros from "trace

Re: [PATCH v3 09/29] plugins: Include trace/mem.h in api.c

2020-01-02 Thread Philippe Mathieu-Daudé
Hi Richard, On 12/29/19 12:11 AM, Richard Henderson wrote: Code movement in an upcoming patch will show that this file was implicitly depending on trace/mem.h being included beforehand. Ah, it uses the TRACE_MEM_* macros from "trace/mem-internal.h", which is include by "trace/mem.h". OK. Wh

[PATCH v3 09/29] plugins: Include trace/mem.h in api.c

2019-12-28 Thread Richard Henderson
Code movement in an upcoming patch will show that this file was implicitly depending on trace/mem.h being included beforehand. Reported-by: Alex Bennée Signed-off-by: Richard Henderson --- plugins/api.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/plugins/api.c b/plugins/api.c index fa