xiezhiheng <xiezhih...@huawei.com> writes: >> -----Original Message----- >> From: Richard Sandiford [mailto:richard.sandif...@arm.com] >> Sent: Tuesday, November 3, 2020 9:57 PM >> To: xiezhiheng <xiezhih...@huawei.com> >> Cc: gcc-patches@gcc.gnu.org >> Subject: Re: [PATCH PR94442] [AArch64] Redundant ldp/stp instructions >> emitted at -O3 >> >> Thanks, I pushed both patches to trunk. >> > > Thanks. And I made two separate patches for these two groups, tbl/tbx > intrinsics and > the rest of the arithmetic operation intrinsics. > > Note: It does not matter which patch is applied first.
I pushed the TBL/TBX one, but on the other patch: > @@ -297,7 +297,7 @@ > BUILTIN_VSDQ_I (USHIFTIMM, uqshl_n, 0, ALL) > > /* Implemented by aarch64_reduc_plus_<mode>. */ > - BUILTIN_VALL (UNOP, reduc_plus_scal_, 10, ALL) > + BUILTIN_VALL (UNOP, reduc_plus_scal_, 10, FP) This is defined for integer and FP modes, so I think it should be NONE instead of FP. We'll automatically add FLAGS_FP based on the mode where necessary. Otherwise it looks good, thanks. Richard