Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-08-29 Thread Richard Biener via Gcc-patches
On Tue, 5 Jul 2022, Richard Sandiford wrote: > Tamar Christina writes: > >> > so that the multiple_p test is skipped if the structure is undefined. > >> > >> Actually, we should probably skip the constant_multiple_p test as well. > >> Keeping it would only be meaningful for little-endian. > >>

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-07-05 Thread Richard Sandiford via Gcc-patches
Tamar Christina writes: >> > so that the multiple_p test is skipped if the structure is undefined. >> >> Actually, we should probably skip the constant_multiple_p test as well. >> Keeping it would only be meaningful for little-endian. >> >> simplify_gen_subreg should alread do the necessary chec

RE: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-07-05 Thread Tamar Christina via Gcc-patches
> > so that the multiple_p test is skipped if the structure is undefined. > > Actually, we should probably skip the constant_multiple_p test as well. > Keeping it would only be meaningful for little-endian. > > simplify_gen_subreg should alread do the necessary checks to make sure > that the subr

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-24 Thread Jeff Law via Gcc-patches
On 6/15/2022 5:36 AM, Richard Sandiford wrote: Jeff Law via Gcc-patches writes: On 6/13/2022 5:54 AM, Richard Biener wrote: On Sun, Jun 12, 2022 at 7:27 PM Jeff Law via Gcc-patches wrote: [...] On a related topic, any thoughts on keeping complex objects as complex types/modes through gimp

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-20 Thread Richard Sandiford via Gcc-patches
Richard Sandiford via Gcc-patches writes: > Tamar Christina writes: >>> -Original Message- >>> From: Richard Sandiford >>> Sent: Monday, June 13, 2022 9:41 AM >>> To: Tamar Christina >>> Cc: gcc-patches@gcc.gnu.org; nd ; rguent...@suse.d

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-17 Thread Richard Sandiford via Gcc-patches
Tamar Christina writes: >> -Original Message- >> From: Richard Sandiford >> Sent: Monday, June 13, 2022 9:41 AM >> To: Tamar Christina >> Cc: gcc-patches@gcc.gnu.org; nd ; rguent...@suse.de >> Subject: Re: [PATCH]middle-end Use subregs to expan

RE: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-16 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Richard Sandiford > Sent: Monday, June 13, 2022 9:41 AM > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; rguent...@suse.de > Subject: Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to > set the lowpart. > > Tama

RE: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-16 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Gcc-patches bounces+tamar.christina=arm@gcc.gnu.org> On Behalf Of Richard > Sandiford via Gcc-patches > Sent: Wednesday, June 15, 2022 12:36 PM > To: Jeff Law via Gcc-patches > Subject: Re: [PATCH]middle-end Use subregs to expand COMP

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-15 Thread Richard Sandiford via Gcc-patches
Jeff Law via Gcc-patches writes: > On 6/13/2022 5:54 AM, Richard Biener wrote: >> On Sun, Jun 12, 2022 at 7:27 PM Jeff Law via Gcc-patches >> wrote: >> [...] >>> On a related topic, any thoughts on keeping complex objects as complex >>> types/modes through gimple and into at least parts of the RT

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-13 Thread Jeff Law via Gcc-patches
On 6/13/2022 4:19 AM, Tamar Christina wrote: -Original Message- From: Gcc-patches On Behalf Of Jeff Law via Gcc-patches Sent: Sunday, June 12, 2022 6:27 PM To: gcc-patches@gcc.gnu.org Subject: Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart. On 6/9/2022

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-13 Thread Jeff Law via Gcc-patches
On 6/13/2022 5:54 AM, Richard Biener wrote: On Sun, Jun 12, 2022 at 7:27 PM Jeff Law via Gcc-patches wrote: [...] On a related topic, any thoughts on keeping complex objects as complex types/modes through gimple and into at least parts of the RTL pipeline? The way complex arithmetic instruc

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-13 Thread Richard Biener via Gcc-patches
On Sun, Jun 12, 2022 at 7:27 PM Jeff Law via Gcc-patches wrote: [...] > On a related topic, any thoughts on keeping complex objects as complex > types/modes through gimple and into at least parts of the RTL pipeline? > > The way complex arithmetic instructions work on our chip is going to be > ext

RE: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-13 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Gcc-patches bounces+tamar.christina=arm@gcc.gnu.org> On Behalf Of Jeff Law via > Gcc-patches > Sent: Sunday, June 12, 2022 6:27 PM > To: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-13 Thread Richard Sandiford via Gcc-patches
Tamar Christina writes: > Hi All, > > When lowering COMPLEX_EXPR we currently emit two VEC_EXTRACTs. One for the > lowpart and one for the highpart. > > The problem with this is that in RTL the lvalue of the RTX is the only thing > tying the two instructions together. > > This means that e.g. com

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-12 Thread Jeff Law via Gcc-patches
On 6/9/2022 1:52 AM, Tamar Christina via Gcc-patches wrote: Hi All, When lowering COMPLEX_EXPR we currently emit two VEC_EXTRACTs. One for the lowpart and one for the highpart. The problem with this is that in RTL the lvalue of the RTX is the only thing tying the two instructions together.