Re: [BUILDROBOT] [Ada] error: alignment of array elements is greater than element size (was: [PATCH] GIMPLE store merging pass)

2016-10-29 Thread Markus Trippelsdorf
On 2016.10.29 at 19:56 +0200, Jan-Benedict Glaw wrote: > Hi Kyrill! > > On Mon, 2016-10-24 15:56:48 +0100, Kyrill Tkachov > wrote: > > This is a slight update over [1] with Richard's feedback addressed. > > In terminate_all_aliasing_chains we now terminate the chain early if > > the destination

[BUILDROBOT] [Ada] error: alignment of array elements is greater than element size (was: [PATCH] GIMPLE store merging pass)

2016-10-29 Thread Jan-Benedict Glaw
Hi Kyrill! On Mon, 2016-10-24 15:56:48 +0100, Kyrill Tkachov wrote: > This is a slight update over [1] with Richard's feedback addressed. > In terminate_all_aliasing_chains we now terminate the chain early if > the destination is writing to a base offset by a variable amount. > This avoids walki

Re: [PATCH] GIMPLE store merging pass

2016-10-28 Thread Kyrill Tkachov
On 27/10/16 14:31, Richard Biener wrote: On Mon, 24 Oct 2016, Kyrill Tkachov wrote: Hi all, This is a slight update over [1] with Richard's feedback addressed. In terminate_all_aliasing_chains we now terminate the chain early if the destination is writing to a base offset by a variable amount

Re: [PATCH] GIMPLE store merging pass

2016-10-27 Thread Richard Biener
On Mon, 24 Oct 2016, Kyrill Tkachov wrote: > Hi all, > > This is a slight update over [1] with Richard's feedback addressed. > In terminate_all_aliasing_chains we now terminate the chain early if > the destination is writing to a base offset by a variable amount. > This avoids walking the store c

[PATCH] GIMPLE store merging pass

2016-10-24 Thread Kyrill Tkachov
Hi all, This is a slight update over [1] with Richard's feedback addressed. In terminate_all_aliasing_chains we now terminate the chain early if the destination is writing to a base offset by a variable amount. This avoids walking the store chain and performing more alias checks. The param max-s