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
. Martin >From 386fae51b258bd51b700596697b223e492960ff0 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Fri, 8 Nov 2019 16:05:00 +0100 Subject: [PATCH] Enhance syntax of -fdbg-cnt. gcc/ChangeLog: 2019-11-08 Martin Liska * common.opt: Document change of -fdbg-cnt option. * dbgcnt.c (DEBUG_COUNTER): Remove.

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

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

2019-11-11 Thread Martin Liška
Nov 2019 16:05:00 +0100 Subject: [PATCH] Enhance syntax of -fdbg-cnt. gcc/ChangeLog: 2019-11-08 Martin Liska * common.opt: Document change of -fdbg-cnt option. * dbgcnt.c (DEBUG_COUNTER): Remove. (dbg_cnt_is_enabled): Remove. (dbg_cnt): Work with new intervals. (dbg_cnt_set_limit_by_in

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

2019-11-11 Thread Martin Liška
Sep 17 00:00:00 2001 From: Martin Liska Date: Fri, 8 Nov 2019 16:05:00 +0100 Subject: [PATCH] Enhance syntax of -fdbg-cnt. gcc/ChangeLog: 2019-11-08 Martin Liska * common.opt: Document change of -fdbg-cnt option. * dbgcnt.c (DEBUG_COUNTER): Remove. (dbg_cnt_is_enabled): Remove. (dbg_cnt): W

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)

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

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

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

[PATCH] Enhance syntax of -fdbg-cnt.

2019-11-11 Thread Martin Liška
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 closed, it's more intuitive. Example: $ g++ -O2 tramp3d-v4.ii