This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] [aarch64][v2] Fix falkor pipeline description for dup<q>


On Thu, Aug 02, 2018 at 11:58:37AM -0500, Siddhesh Poyarekar wrote:
> There was a typo in the pipeline description where DUP was assigned to
> the vector pipes for quad mode ops when it really only uses the VTOG
> pipes.  Fixing this does not show any noticeable difference in
> performance (there's a very small bump of 1.7% in x264 but that's
> about it) in my tests but is the more precise description of operations
> for falkor.
> 
> Bootstrapped and tested with --with-cpu=falkor to confirm that there
> are no regressions.

OK.

Thanks,
James

> 
> Siddhesh
> 
> Changes from v1:
> 
> - Fixed up ChangeLog entry
> 
> - Replaced falkor_gtov,falkor_gtov with falkor_gtov*2
> 
> 	* config/aarch64/falkor.md (falkor_am_1_vxvy_vxvy): Move
> 	neon_dup_q to...
> 	(falkor_am_1_gtov_gtov): ... a new insn reservation.
> 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]