Re: [PATCH v6] genemit.c (main): split insn-emit.c for compiling parallelly

2020-09-15 Thread Jojo R
Hi, Ok & Thanks, I will do my best for it :) It’s fixed in patch v7. Jojo 在 2020年9月11日 +0800 PM11:29,Segher Boessenkool ,写道: > Hi! > > On Fri, Sep 11, 2020 at 03:26:17PM +0800, Jojo R wrote: > > +#define printf_include() do { \ > > Don't use macros please, use a function? > > And

Re: [PATCH v6] genemit.c (main): split insn-emit.c for compiling parallelly

2020-09-11 Thread Segher Boessenkool
Hi! On Fri, Sep 11, 2020 at 03:26:17PM +0800, Jojo R wrote: > +#define printf_include() do { \ Don't use macros please, use a function? And maybe do this in a separate patch, for ease of review. That should be ack'ed pretty much immediately, after which it is out of the way, and we do not have

[PATCH v6] genemit.c (main): split insn-emit.c for compiling parallelly

2020-09-11 Thread Jojo R
gcc/ChangeLog: * genemit.c (main): Print 'split line'. * Makefile.in (insn-emit.c): Define split count and file --- gcc/Makefile.in | 19 +++ gcc/genemit.c | 128 ++-- 2 files changed, 89 insertions(+), 58 deletions(-) diff --gi