On Wed, Jan 4, 2017 at 10:15 PM, Jakub Jelinek <ja...@redhat.com> wrote: > On Wed, Jan 04, 2017 at 01:31:28PM +0100, Richard Biener wrote: >> > And not sure why this actually >> > is RejectNegative, wouldn't >> > Common Alias(fvect-cost-model=,dynamic,unlimited) >> > work just on fvect-cost-model (can test that)? >> >> Good question ;) If it works, ok. > > And here is a patch for that, in addition to bootstrap/regtests I've > tried in the debugger the effect of both -fvect-cost-model and > -fno-vect-cost-model and they set global_options.x_flag_vect_cost_model > to the expected enum values in each case. > > Ok for trunk?
Ok. Richard. > 2017-01-04 Jakub Jelinek <ja...@redhat.com> > > * common.opt (fvect-cost-model): Remove RejectNegative flag, use > 3 argument Alias with unlimited for the negative form. > (fno-vect-cost-model): Removed. > > --- gcc/common.opt.jj 2017-01-01 12:45:37.000000000 +0100 > +++ gcc/common.opt 2017-01-04 18:30:32.239318711 +0100 > @@ -2706,13 +2706,9 @@ EnumValue > Enum(vect_cost_model) String(cheap) Value(VECT_COST_MODEL_CHEAP) > > fvect-cost-model > -Common RejectNegative Alias(fvect-cost-model=,dynamic) > +Common Alias(fvect-cost-model=,dynamic,unlimited) > Enables the dynamic vectorizer cost model. Preserved for backward > compatibility. > > -fno-vect-cost-model > -Common RejectNegative Alias(fvect-cost-model=,unlimited) > -Enables the unlimited vectorizer cost model. Preserved for backward > compatibility. > - > ftree-vect-loop-version > Common Ignore > Does nothing. Preserved for backward compatibility. > > > Jakub
