On Fri, Sep 30, 2016 at 6:57 PM, Kyrill Tkachov <kyrylo.tkac...@foss.arm.com> wrote: > Hi Richard, > > On 29/09/16 11:45, Richard Biener wrote: >> >> >> + gimple_seq seq = NULL; >> + unsigned int num_stmts = 0; >> + tree offset_type = get_type_for_merged_store (group); >> + tree last_vdef, new_vuse; >> + last_vdef = gimple_vdef (group->last_stmt); >> + new_vuse = gimple_vuse (group->last_stmt); >> + location_t loc = get_merged_store_location (group); >> If you end up splitting the store then please use a location appropriate >> for the split part. Likewise for the alias type. >> > > How would I get the appropriate alias type? > Is there some way to construct it from the alias type of the base > object offset by some number of bytes?
You already have a function for this - it just looks at the whole group instead of a subgroup. As said, if you'd first split groups then then do the emission this part would resolve itself. Richard. > > Thanks, > Kyrill