[PATCH 1/3] Delay location expansion within rich_location until printing

2015-12-11 Thread David Malcolm
Previously, source_location/location_t values passed to rich_location were immediately expanded (to expanded_location instances stored inside the rich_location). This patch updates the insides of class rich_location to delay this expansion until the insides of diagnostic_show_locus. This

[PATCH 3/3] PR c/68473: sanitize source range-printing within certain macro expansions (v2)

2015-12-11 Thread David Malcolm
Changes in v2: - add two more testcases, based on additional reproducers reported (one to the PR, another to a dup); now adds 17 PASS results to gcc.sum. - update for delayed expansion of locations within rich_location and the terminology fixes from the other patches in this kit.

[PATCH 0/3] Re: [PATCH] PR c/68473: sanitize source range-printing within certain macro expansions

2015-12-11 Thread David Malcolm
On Tue, 2015-11-24 at 13:43 +0100, Bernd Schmidt wrote: > On 11/23/2015 07:26 PM, David Malcolm wrote: > > > > In theory we could attempt to try to handle this kind of thing by > > looking at the macro expansions, and to print something like: > > > > 13 TEST_EQ (fmin); > >

Re: RFA (hash-*): PATCH for c++/68309

2015-12-11 Thread Jason Merrill
On 12/11/2015 05:10 AM, Richard Biener wrote: On Thu, Dec 10, 2015 at 11:03 PM, Jason Merrill wrote: The C++ front end uses a temporary hash table to remember specializations of local variables during template instantiations. In a nested function such as a lambda or local

Re: [PATCH] Fix ICE with -fno-if-conversion (PR rtl-optimization/68730)

2015-12-11 Thread Jeff Law
On 12/11/2015 01:13 PM, Jakub Jelinek wrote: On Fri, Dec 11, 2015 at 07:30:59AM +0100, Richard Biener wrote: So, to fix ICE on the following testcase, we can either do what the patch does, or could conditionalize both the calculate_dominance_info and free_dominance_info in the

<    1   2