Re: [PATCH 1/2] PR c/68757: fix uninitialied src_range for various builtins

2015-12-08 Thread Bernd Schmidt
On 12/08/2015 05:02 PM, David Malcolm wrote: I actually implemented something like this when implementing these two patches. Work-in-progress patch attached, which introduces an INVALID_LOCATION value for source_location/location_t, and uses it to "poison" the initial value of c_expr's src_rang

Re: [PATCH 1/2] PR c/68757: fix uninitialied src_range for various builtins

2015-12-08 Thread David Malcolm
On Tue, 2015-12-08 at 16:38 +0100, Bernd Schmidt wrote: > On 12/08/2015 04:43 PM, David Malcolm wrote: > > This fixes various uninitialized src_range of c_expr, this time > > in the various builtins that are parsed via c_parser_get_builtin_args. > > > > Bootstrapped®rtested on x86_64-pc-linux-gnu.

Re: [PATCH 1/2] PR c/68757: fix uninitialied src_range for various builtins

2015-12-08 Thread Bernd Schmidt
On 12/08/2015 04:43 PM, David Malcolm wrote: This fixes various uninitialized src_range of c_expr, this time in the various builtins that are parsed via c_parser_get_builtin_args. Bootstrapped®rtested on x86_64-pc-linux-gnu. OK for trunk? I think both of these patches are OK. Some questions t

[PATCH 1/2] PR c/68757: fix uninitialied src_range for various builtins

2015-12-08 Thread David Malcolm
This fixes various uninitialized src_range of c_expr, this time in the various builtins that are parsed via c_parser_get_builtin_args. Bootstrapped®rtested on x86_64-pc-linux-gnu. OK for trunk? gcc/c/ChangeLog: PR c/68757 * c-parser.c (c_parser_get_builtin_args): Add "out