Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-12 Thread Kyrill Tkachov
On 10/10/16 12:15, Richard Biener wrote: On Mon, 10 Oct 2016, Richard Biener wrote: On Mon, 10 Oct 2016, Kyrill Tkachov wrote: On 10/10/16 11:22, Richard Biener wrote: On Mon, 10 Oct 2016, Kyrill Tkachov wrote: Hi Richard, As I mentioned, here is the patch applying to the main store merg

Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Richard Biener
On Mon, 10 Oct 2016, Richard Biener wrote: > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > > > On 10/10/16 11:22, Richard Biener wrote: > > > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > > > > > Hi Richard, > > > > > > > > As I mentioned, here is the patch applying to the main store mer

Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Richard Biener
On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > On 10/10/16 11:22, Richard Biener wrote: > > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > > > Hi Richard, > > > > > > As I mentioned, here is the patch applying to the main store merging patch > > > to > > > re-implement encode_tree_to_bitpos >

Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Kyrill Tkachov
On 10/10/16 12:06, Kyrill Tkachov wrote: On 10/10/16 11:22, Richard Biener wrote: On Mon, 10 Oct 2016, Kyrill Tkachov wrote: Hi Richard, As I mentioned, here is the patch applying to the main store merging patch to re-implement encode_tree_to_bitpos to operate on the bytes directly. This w

Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Kyrill Tkachov
On 10/10/16 11:22, Richard Biener wrote: On Mon, 10 Oct 2016, Kyrill Tkachov wrote: Hi Richard, As I mentioned, here is the patch applying to the main store merging patch to re-implement encode_tree_to_bitpos to operate on the bytes directly. This works fine on little-endian but breaks on bi

Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Richard Biener
On Mon, 10 Oct 2016, Richard Biener wrote: > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > Hi Richard, > > > > As I mentioned, here is the patch applying to the main store merging patch > > to > > re-implement encode_tree_to_bitpos > > to operate on the bytes directly. > > > > This works fi

Re: [PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Richard Biener
On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > Hi Richard, > > As I mentioned, here is the patch applying to the main store merging patch to > re-implement encode_tree_to_bitpos > to operate on the bytes directly. > > This works fine on little-endian but breaks on big-endian, even for merging > bi

[PATCH][store merging][RFA] Re-implement merging code

2016-10-10 Thread Kyrill Tkachov
Hi Richard, As I mentioned, here is the patch applying to the main store merging patch to re-implement encode_tree_to_bitpos to operate on the bytes directly. This works fine on little-endian but breaks on big-endian, even for merging bitfields within a single byte. Consider the code snippet f