> -----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, 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. +
store-v2.patch
Description: store-v2.patch