Re: [patch] Small improvements to coverage info (1/n)

2019-07-05 Thread Eric Botcazou
> This change broke gomp/pr88107.c test: > FAIL: gcc.dg/gomp/pr88107.c (internal compiler error) > FAIL: gcc.dg/gomp/pr88107.c (test for excess errors) > Excess errors: > during GIMPLE pass: graphite > /usr/src/gcc/gcc/testsuite/gcc.dg/gomp/pr88107.c:26:1: internal compiler > error: Segmentation fa

Re: [patch] Small improvements to coverage info (1/n)

2019-07-05 Thread Jeff Law
On 7/5/19 6:12 AM, Jakub Jelinek wrote: > On Wed, Jul 03, 2019 at 12:46:37PM +0200, Eric Botcazou wrote: >> 2019-07-03 Eric Botcazou >> >> * tree-cfg.c (gimple_make_forwarder_block): Propagate location info on >> phi nodes if possible. >> * tree-scalar-evolution.c (final_value_rep

Re: [patch] Small improvements to coverage info (1/n)

2019-07-05 Thread Jakub Jelinek
On Wed, Jul 03, 2019 at 12:46:37PM +0200, Eric Botcazou wrote: > 2019-07-03 Eric Botcazou > > * tree-cfg.c (gimple_make_forwarder_block): Propagate location info on > phi nodes if possible. > * tree-scalar-evolution.c (final_value_replacement_loop): Propagate > location

Re: [patch] Small improvements to coverage info (1/n)

2019-07-03 Thread Jeff Law
On 7/3/19 4:46 AM, Eric Botcazou wrote: > Hi, > > we have collected a number of small improvements to coverage info generated > by > the compiler over the years. One of the issues is when a new expression or > statement is built without source location information and ends up inheriting > the

[patch] Small improvements to coverage info (1/n)

2019-07-03 Thread Eric Botcazou
Hi, we have collected a number of small improvements to coverage info generated by the compiler over the years. One of the issues is when a new expression or statement is built without source location information and ends up inheriting the source location information of the previous instructio