Re: [PATCH 2/5 v3] Vect peeling cost model

2017-05-24 Thread Richard Sandiford
Robin Dapp writes: >> Not sure I've understood the series TBH, but is the npeel == vf / 2 >> there specifically for the "unknown number of peels" case? How do >> we distinguish that from the case in which the number of peels is >> known to be vf / 2 at compile time? Or have I missed the point >>

Re: [PATCH 2/5 v3] Vect peeling cost model

2017-05-24 Thread Robin Dapp
> Not sure I've understood the series TBH, but is the npeel == vf / 2 > there specifically for the "unknown number of peels" case? How do > we distinguish that from the case in which the number of peels is > known to be vf / 2 at compile time? Or have I missed the point > completely? (probably ye

Re: [PATCH 2/5 v3] Vect peeling cost model

2017-05-23 Thread Richard Sandiford
Robin Dapp writes: > @@ -1272,12 +1272,40 @@ vect_peeling_hash_get_lowest_cost (_vect_peel_info > **slot, > && !STMT_VINFO_GROUPED_ACCESS (stmt_info)) > continue; > > + int save_misalignment; >save_misalignment = DR_MISALIGNMENT (dr); > - vect_update_misalignment

[PATCH 2/5 v3] Vect peeling cost model

2017-05-23 Thread Robin Dapp
gcc/ChangeLog: 2017-05-23 Robin Dapp * tree-vect-data-refs.c (vect_update_misalignment_for_peel): Rename. (vect_get_peeling_costs_all_drs): Create function. (vect_peeling_hash_get_lowest_cost): Use vect_get_peeling_costs_all_drs. (vect_peeling_su