Re: [PATCH] v2: Add IntegerRange for -param=min-nondebug-insn-uid= and fix vector growing in LRA and vec [PR112411]

2023-12-08 Thread Vladimir Makarov
On 12/7/23 03:39, Jakub Jelinek wrote: On Thu, Dec 07, 2023 at 09:36:22AM +0100, Jakub Jelinek wrote: So, one way to fix the LRA issue would be just to use lra_insn_recog_data_len = index * 3U / 2; if (lra_insn_recog_data_len <= index) lra_insn_recog_data_len = index + 1; basically

Re: [PATCH] v2: Add IntegerRange for -param=min-nondebug-insn-uid= and fix vector growing in LRA and vec [PR112411]

2023-12-07 Thread Richard Biener
On Thu, 7 Dec 2023, Jakub Jelinek wrote: > On Thu, Dec 07, 2023 at 11:12:39AM +0100, Richard Biener wrote: > > > 2023-12-07 Jakub Jelinek > > > > > > PR middle-end/112411 > > > * params.opt (-param=min-nondebug-insn-uid=): Add > > > IntegerRange(0, 1073741824). > > > * lra.cc (check_an

Re: [PATCH] v2: Add IntegerRange for -param=min-nondebug-insn-uid= and fix vector growing in LRA and vec [PR112411]

2023-12-07 Thread Jakub Jelinek
On Thu, Dec 07, 2023 at 11:12:39AM +0100, Richard Biener wrote: > > 2023-12-07 Jakub Jelinek > > > > PR middle-end/112411 > > * params.opt (-param=min-nondebug-insn-uid=): Add > > IntegerRange(0, 1073741824). > > * lra.cc (check_and_expand_insn_recog_data): Use 3U rather than 3

Re: [PATCH] v2: Add IntegerRange for -param=min-nondebug-insn-uid= and fix vector growing in LRA and vec [PR112411]

2023-12-07 Thread Richard Biener
On Thu, 7 Dec 2023, Jakub Jelinek wrote: > On Thu, Dec 07, 2023 at 09:36:22AM +0100, Jakub Jelinek wrote: > > So, one way to fix the LRA issue would be just to use > > lra_insn_recog_data_len = index * 3U / 2; > > if (lra_insn_recog_data_len <= index) > > lra_insn_recog_data_len = index +

[PATCH] v2: Add IntegerRange for -param=min-nondebug-insn-uid= and fix vector growing in LRA and vec [PR112411]

2023-12-07 Thread Jakub Jelinek
On Thu, Dec 07, 2023 at 09:36:22AM +0100, Jakub Jelinek wrote: > So, one way to fix the LRA issue would be just to use > lra_insn_recog_data_len = index * 3U / 2; > if (lra_insn_recog_data_len <= index) > lra_insn_recog_data_len = index + 1; > basically do what vec.cc does. I thought we ca