Re: [PATCH] Fix up flag_cunroll_grow_size handling in presence of optimize attr [PR96535]

2020-08-12 Thread Segher Boessenkool
Hi Jakub, On Wed, Aug 12, 2020 at 10:20:12AM +0200, Jakub Jelinek wrote: > This patch moves the unrolling related handling from process_options into > finish_options which is invoked whenever the options are being finalized, > and the rs6000 specific parts into the override_options_after_change ho

Re: [PATCH] Fix up flag_cunroll_grow_size handling in presence of optimize attr [PR96535]

2020-08-12 Thread Richard Biener
On August 12, 2020 10:20:12 AM GMT+02:00, Jakub Jelinek wrote: >Hi! > >As the testcase in the PR shows (not included in the patch, as >it seems quite fragile to observe unrolling in the IL), the >introduction of >flag_cunroll_grow_size broke optimize attribute related to loop >unrolling. >The pro

[PATCH] Fix up flag_cunroll_grow_size handling in presence of optimize attr [PR96535]

2020-08-12 Thread Jakub Jelinek via Gcc-patches
Hi! As the testcase in the PR shows (not included in the patch, as it seems quite fragile to observe unrolling in the IL), the introduction of flag_cunroll_grow_size broke optimize attribute related to loop unrolling. The problem is that the new option flag is set (if not set explicitly) only in p