Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-07 Thread David Malcolm
On Fri, 2014-11-07 at 12:47 -0700, Jeff Law wrote: > On 11/05/14 12:34, David Malcolm wrote: > > > > > I've added comments throughout the file. > > > > I didn't bother adding __attribute__((cold)), instead simply dropping > > that "TODO". > Fine. > > > > > Attached is the current state of the fil

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-07 Thread Jeff Law
On 11/05/14 12:34, David Malcolm wrote: I've added comments throughout the file. I didn't bother adding __attribute__((cold)), instead simply dropping that "TODO". Fine. Attached is the current state of the file gcc/jit/libgccjit.c (on the branch) for review. OK for trunk? (conditional on

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: > On 11/04/14 09:57, David Malcolm wrote: > >>> +#define IS_ASCII_DIGIT(CHAR) \ > >>> + ((CHAR) >= '0' && (CHAR) <='9') > >>> + > >>> +#define IS_ASCII_ALNUM(CHAR) \ > >>> + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) > >> Can't we rely on t

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-04 Thread Jeff Law
On 11/04/14 09:57, David Malcolm wrote: +#define IS_ASCII_DIGIT(CHAR) \ + ((CHAR) >= '0' && (CHAR) <='9') + +#define IS_ASCII_ALNUM(CHAR) \ + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) Can't we rely on the C library to give us equivalents? I've been burned in the past by the C library

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-04 Thread David Malcolm
On Mon, 2014-11-03 at 13:32 -0700, Jeff Law wrote: > On 10/31/14 11:02, David Malcolm wrote: > > This file implements the entrypoints of the library's public API. > > > > It performs error-checking at this boundary, before calling into the > > jit-recording.h internal API. > > > > gcc/jit/ > >

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-03 Thread Jeff Law
On 10/31/14 11:02, David Malcolm wrote: This file implements the entrypoints of the library's public API. It performs error-checking at this boundary, before calling into the jit-recording.h internal API. gcc/jit/ * libgccjit.c: New. --- gcc/jit/libgccjit.c | 1506