xiezhiheng <xiezhih...@huawei.com> writes: >> -----Original Message----- >> From: Richard Sandiford [mailto:richard.sandif...@arm.com] >> Sent: Wednesday, October 21, 2020 12:54 AM >> To: xiezhiheng <xiezhih...@huawei.com> >> Cc: Richard Biener <richard.guent...@gmail.com>; gcc-patches@gcc.gnu.org >> Subject: Re: [PATCH PR94442] [AArch64] Redundant ldp/stp instructions >> emitted at -O3 >> >> xiezhiheng <xiezhih...@huawei.com> writes: >> > I made two separate patches for these two groups, get/set register >> intrinsics and store intrinsics. >> > >> > Note: It does not matter which patch is applied first. >> > >> > Bootstrapped and tested on aarch64 Linux platform. >> >> Thanks. I pushed the get/set patch. For the store patch, I think >> we should have: >> >> const unsigned int FLAG_STORE = FLAG_WRITE_MEMORY | FLAG_AUTO_FP; >> >> since the FP forms don't (for example) read the FPCR. >> > > That's true. I added FLAG_STORE for the store intrinsics and made the patch > for them. > > Bootstrapped and tested on aarch64 Linux platform.
Thanks, pushed to trunk. Sorry for the delayed response. Richard > > Thanks, > Xie Zhiheng > > > diff --git a/gcc/ChangeLog b/gcc/ChangeLog > index 59fa1ad4d5d..26edaa309c8 100644 > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,10 @@ > +2020-10-22 Zhiheng Xie <xiezhih...@huawei.com> > + Nannan Zheng <zhengnan...@huawei.com> > + > + * config/aarch64/aarch64-builtins.c: Add FLAG STORE. > + * config/aarch64/aarch64-simd-builtins.def: Add proper FLAG > + for store intrinsics. > +
