Re: [C PATCH] Improve column info of initializers (PR c/60114)

[Jeff Law <law at redhat dot com>]

On 02/10/14 06:44, Marek Polacek wrote:
> This patch improves location information in a bunch of various
> initializers; see the testcase.  Main issue was that digest_init
> was getting only input_location.
>
> Regtested/bootstrapped on x86_64-linux, ok for trunk at this stage
> or should I queue it for 5.0?
>
> 2014-02-10  Marek Polacek  <polacek@redhat.com>
>
> 	PR c/60114
> c/
> 	* c-parser.c (c_parser_initelt): Pass input_location to
> 	process_init_element.
> 	(c_parser_initval): Pass loc to process_init_element.
> 	* c-tree.h (process_init_element): Adjust declaration.
> 	* c-typeck.c (push_init_level): Pass input_location to
> 	process_init_element.
> 	(pop_init_level): Likewise.
> 	(set_designator): Likewise.
> 	(output_init_element): Add location_t parameter.  Pass loc to
> 	digest_init.
> 	(output_pending_init_elements): Pass input_location to
> 	output_init_element.
> 	(process_init_element): Add location_t parameter.  Pass loc to
> 	output_init_element.
> testsuite/
> 	* gcc.dg/pr60114.c: New test.
I'd think this probably should queue for the next stage1.  We're 
probably just ~6wks away at this point anyway.


jeff