Hi Bill, On Wed, Jun 26, 2019 at 10:23:06PM -0500, Bill Schmidt wrote: > 2019-06-27 Bill Schmidt <wschm...@linux.ibm.com> > > * config/rs6000/rs6000.c (rs6000_option_override_internal): Enable > -fvariable-expansion-in-unroller by default. > > > --- gcc/config/rs6000/rs6000.c (revision 272719) > +++ gcc/config/rs6000/rs6000.c (working copy) > @@ -3616,6 +3616,11 @@ rs6000_option_override_internal (bool global_init_ > && !global_options_set.x_flag_asynchronous_unwind_tables) > flag_asynchronous_unwind_tables = 1; > > + /* -fvariable-expansion-in-unroller is a win for POWER whenever the > + loop unroller is active. It is only checked during unrolling, so > + we can just set it on by default. */ > + flag_variable_expansion_in_unroller = 1;
You should test global_options_set like the asynch unwind thing above, so that a user can still turn off variable expansion if he/she so desires. Okay with that change. Thanks! Segher
