Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-05-06 Thread Segher Boessenkool
Hi! On Tue, May 05, 2020 at 08:41:35PM -0400, Hans-Peter Nilsson wrote: > On Fri, 1 May 2020, Segher Boessenkool wrote: > > On Mon, Apr 27, 2020 at 05:01:34PM -0500, will schmidt wrote: > > > On Mon, 2020-04-27 at 15:53 -0400, Michael Meissner via Gcc-patches wrote: > > > > +unsigned long > > > >

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-05-05 Thread Hans-Peter Nilsson
On Fri, 1 May 2020, Segher Boessenkool wrote: > On Mon, Apr 27, 2020 at 05:01:34PM -0500, will schmidt wrote: > > On Mon, 2020-04-27 at 15:53 -0400, Michael Meissner via Gcc-patches wrote: > > > +unsigned long > > > +load_us_offset1 (unsigned char *p) > > > +{ > > > + return *(unsigned short *)(p

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-05-01 Thread will schmidt via Gcc-patches
On Fri, 2020-05-01 at 17:02 -0500, Segher Boessenkool wrote: > On Fri, May 01, 2020 at 03:54:26PM -0500, will schmidt wrote: > > > The other way around :-) stfs is for single precision float > > > ("float", > > > in C), while stfd is for double precision float ("double", in C). > > > > I came up

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-05-01 Thread Segher Boessenkool
On Fri, May 01, 2020 at 03:54:26PM -0500, will schmidt wrote: > > The other way around :-) stfs is for single precision float > > ("float", > > in C), while stfd is for double precision float ("double", in C). > > I came up with my comment based on what was being tested for further > below. If

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-05-01 Thread will schmidt via Gcc-patches
On Fri, 2020-05-01 at 10:48 -0500, Segher Boessenkool wrote: > Hi! > > On Mon, Apr 27, 2020 at 05:01:34PM -0500, will schmidt wrote: > > On Mon, 2020-04-27 at 15:53 -0400, Michael Meissner via Gcc-patches > > wrote: > > > This patch adds a test that verifies that the compiler generates > > > a pr

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-05-01 Thread Segher Boessenkool
Hi! On Mon, Apr 27, 2020 at 05:01:34PM -0500, will schmidt wrote: > On Mon, 2020-04-27 at 15:53 -0400, Michael Meissner via Gcc-patches wrote: > > This patch adds a test that verifies that the compiler generates a prefixed > > load/store instruction where the compiler cannot generate the instructi

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-04-27 Thread will schmidt via Gcc-patches
On Mon, 2020-04-27 at 15:53 -0400, Michael Meissner via Gcc-patches wrote: > This patch adds a test that verifies that the compiler generates a prefixed > load/store instruction where the compiler cannot generate the instruction > directly because the offset is not a valid DS or DQ offset. A DS of

[PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

2020-04-27 Thread Michael Meissner via Gcc-patches
This patch adds a test that verifies that the compiler generates a prefixed load/store instruction where the compiler cannot generate the instruction directly because the offset is not a valid DS or DQ offset. A DS offset must have the bottom 2 bits clear. A DQ offset must have the bottom 4 bits