Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-13 Thread Richard Biener
On Wed, Nov 13, 2019 at 10:15 AM Martin Liška wrote: > > On 11/12/19 10:00 AM, Martin Liška wrote: > > There's one another version that I'm testing right now. > > It survives regression tests and bootstrap. > > Richi, may I install it the patch? Yes. Thanks, Richard. > > Thanks, > Martin

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-13 Thread Martin Liška
On 11/12/19 10:00 AM, Martin Liška wrote: There's one another version that I'm testing right now. It survives regression tests and bootstrap. Richi, may I install it the patch? Thanks, Martin

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-12 Thread Martin Liška
On 11/12/19 9:35 AM, Richard Biener wrote: On Mon, Nov 11, 2019 at 3:56 PM Martin Liška wrote: On 11/11/19 3:19 PM, Richard Biener wrote: -static unsigned int limit_low[debug_counter_number_of_counters]; +static auto_vec *limits[debug_counter_number_of_counters] = {NULL}; Hm, apparently it'

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-12 Thread Richard Biener
On Mon, Nov 11, 2019 at 3:56 PM Martin Liška wrote: > > On 11/11/19 3:19 PM, Richard Biener wrote: > > -static unsigned int limit_low[debug_counter_number_of_counters]; > > +static auto_vec > > *limits[debug_counter_number_of_counters] = {NULL}; > > Hm, apparently it's not working. I see a stack c

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Martin Liška
On 11/11/19 4:25 PM, Martin Liška wrote: I'm going to test the patch. Martin There's one more version where I use more references to mitigate indirection of 'counter' vectors. Martin >From e3b8b3edfed1b5ba320d0fe85686908c5c37c22a Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Fri, 8 Nov 2

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Martin Liška
On 11/11/19 3:50 PM, Richard Biener wrote: On Mon, Nov 11, 2019 at 3:33 PM Martin Liška wrote: On 11/11/19 3:19 PM, Richard Biener wrote: On Mon, Nov 11, 2019 at 9:17 AM Martin Liška wrote: Hi. The patch makes debug counter more usable. In particular, one can now list multiple closed inte

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Martin Liška
On 11/11/19 3:19 PM, Richard Biener wrote: -static unsigned int limit_low[debug_counter_number_of_counters]; +static auto_vec *limits[debug_counter_number_of_counters] = {NULL}; Hm, apparently it's not working. I see a stack corruption when calling dbgcnt. I also explicitly called .create (2) f

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Richard Biener
On Mon, Nov 11, 2019 at 3:33 PM Martin Liška wrote: > > On 11/11/19 3:19 PM, Richard Biener wrote: > > On Mon, Nov 11, 2019 at 9:17 AM Martin Liška wrote: > >> > >> Hi. > >> > >> The patch makes debug counter more usable. In particular, one can now > >> list multiple closed intervals and -fdbg-cn

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Martin Liška
On 11/11/19 3:19 PM, Richard Biener wrote: On Mon, Nov 11, 2019 at 9:17 AM Martin Liška wrote: Hi. The patch makes debug counter more usable. In particular, one can now list multiple closed intervals and -fdbg-cnt-list can reflect that. Based on the discussion with Richard, I decided to leave

Re: [PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Richard Biener
On Mon, Nov 11, 2019 at 9:17 AM Martin Liška wrote: > > Hi. > > The patch makes debug counter more usable. In particular, one can now > list multiple closed intervals and -fdbg-cnt-list can reflect that. > Based on the discussion with Richard, I decided to leave semi-closed > intervals and make it