Re: [PATCH, aarch64] Add prefetch support

2015-01-13 Thread Dr. Philipp Tomsich
Great. I should have an update patch-set ready & tested later tonight. Best, Phil. > On 13 Jan 2015, at 15:18, Andrew Pinski wrote: > > On Tue, Jan 13, 2015 at 6:13 AM, Marcus Shawcroft > wrote: >> On 11 January 2015 at 02:37, Andrew Pinski wrote: >>> On Tue, Nov 11, 2014 at 6:47 AM, Marcus S

Re: [PATCH, aarch64] Add prefetch support

2015-01-13 Thread Andrew Pinski
On Tue, Jan 13, 2015 at 6:13 AM, Marcus Shawcroft wrote: > On 11 January 2015 at 02:37, Andrew Pinski wrote: >> On Tue, Nov 11, 2014 at 6:47 AM, Marcus Shawcroft >> wrote: >>> On 30 October 2014 08:54, Gopalasubramanian, Ganesh >>> wrote: >>> 2014-10-30 Ganesh Gopalasubramanian >>> >>> C

Re: [PATCH, aarch64] Add prefetch support

2015-01-13 Thread Marcus Shawcroft
On 11 January 2015 at 02:37, Andrew Pinski wrote: > On Tue, Nov 11, 2014 at 6:47 AM, Marcus Shawcroft > wrote: >> On 30 October 2014 08:54, Gopalasubramanian, Ganesh >> wrote: >> >>> 2014-10-30 Ganesh Gopalasubramanian >> >> Check the whitespace in your ChangeLog line. >> >>> * config/

Re: [PATCH, aarch64] Add prefetch support

2015-01-10 Thread Andrew Pinski
On Tue, Nov 11, 2014 at 6:47 AM, Marcus Shawcroft wrote: > On 30 October 2014 08:54, Gopalasubramanian, Ganesh > wrote: > >> 2014-10-30 Ganesh Gopalasubramanian > > Check the whitespace in your ChangeLog line. > >> * config/arm/types.md (define_attr "type"): Add prefetch. > > The existi

Re: [PATCH, aarch64] Add prefetch support

2014-12-03 Thread Marcus Shawcroft
On 1 December 2014 at 07:48, Gopalasubramanian, Ganesh wrote: > Please ignore the previous patch sent. The attachment was wrong. > >> There's no point in the buffer or the sprintf. >> The text is short enough to repeat whole pattern in the array: > > Updated the patch for the above suggestions. >

Re: [PATCH, aarch64] Add prefetch support

2014-12-01 Thread Richard Henderson
On 12/01/2014 05:48 PM, Gopalasubramanian, Ganesh wrote: > +2014-12-01 Ganesh Gopalasubramanian > + > + * config/aarch64/aarch64.md (define_insn "prefetch"): New. > + Looks good to me. r~

RE: [PATCH, aarch64] Add prefetch support

2014-11-30 Thread Gopalasubramanian, Ganesh
Please ignore the previous patch sent. The attachment was wrong. > There's no point in the buffer or the sprintf. > The text is short enough to repeat whole pattern in the array: Updated the patch for the above suggestions. make -k check RUNTESTFLAGS="execute.exp compile.exp dg.exp" passes. Is i

RE: [PATCH, aarch64] Add prefetch support

2014-11-30 Thread Gopalasubramanian, Ganesh
> There's no point in the buffer or the sprintf. > The text is short enough to repeat whole pattern in the array: Updated the patch for the above suggestions. Is it ok for upstream? Regards Ganesh prefetch.diff Description: prefetch.diff

Re: [PATCH, aarch64] Add prefetch support

2014-11-17 Thread Richard Henderson
On 11/14/2014 09:11 PM, Gopalasubramanian, Ganesh wrote: > +const char * pftype[2][10] > + = { {"PLDL1STRM", "PLDL3KEEP", "PLDL2KEEP", "PLDL1KEEP"}, > + {"PSTL1STRM", "PSTL3KEEP", "PSTL2KEEP", "PSTL1KEEP"}, > + }; The array should be static const char * const pftype[2][4] I'

RE: [PATCH, aarch64] Add prefetch support

2014-11-14 Thread Gopalasubramanian, Ganesh
> For this prefetch patch I suggest we go with the existing "load1". I have removed the changes done in types.md. > The inline patch has been munged by your mailer, I tried applying the patch > to my tree but it is full of escape sequences. Can you either fix your > mailer or submit patches as

Re: [PATCH, aarch64] Add prefetch support

2014-11-11 Thread Marcus Shawcroft
On 30 October 2014 08:54, Gopalasubramanian, Ganesh wrote: > 2014-10-30 Ganesh Gopalasubramanian Check the whitespace in your ChangeLog line. > * config/arm/types.md (define_attr "type"): Add prefetch. The existing schedulers use 'load1'. We can of course split that into two introdu

FW: [PATCH, aarch64] Add prefetch support

2014-11-10 Thread Gopalasubramanian, Ganesh
PING! I am worried if it goes in stage-1. -Original Message- From: Gopalasubramanian, Ganesh Sent: Thursday, October 30, 2014 2:24 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH, aarch64] Add prefetch support Hi, Below is the patch that implements prefetching support. This patch has

[PATCH, aarch64] Add prefetch support

2014-10-30 Thread Gopalasubramanian, Ganesh
Hi, Below is the patch that implements prefetching support. This patch has been already discussed on a) https://gcc.gnu.org/ml/gcc-patches/2014-02/msg01644.html b) https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00612.html I have not added a test as there are ample tests in compile and execute su

Re: [PATCH, aarch64] Add prefetch support

2014-07-09 Thread Richard Earnshaw
First of all, the recognized interval between pings is a week; please don't ping more often than that. On 04/07/14 11:57, Gopalasubramanian, Ganesh wrote: > Hi, > > Attached is a patch that implements > * Prefetch with immediate offset in the range 0 to 32760 (multiple of 8). > Added a predi

FW: [PATCH, aarch64] Add prefetch support

2014-07-08 Thread Gopalasubramanian, Ganesh
PING! -Original Message- From: Gopalasubramanian, Ganesh Sent: Sunday, July 06, 2014 2:12 AM To: gcc-patches@gcc.gnu.org Cc: marcus.shawcr...@arm.com; richard.earns...@arm.com Subject: RE: [PATCH, aarch64] Add prefetch support PING! From

RE: [PATCH, aarch64] Add prefetch support

2014-07-05 Thread Gopalasubramanian, Ganesh
PING! From: Gopalasubramanian, Ganesh Sent: Friday, July 04, 2014 5:57 AM To: gcc-patches@gcc.gnu.org Cc: marcus.shawcr...@arm.com; richard.earns...@arm.com Subject: [PATCH, aarch64] Add prefetch support Hi, Attached is a patch that implements

[PATCH, aarch64] Add prefetch support

2014-07-04 Thread Gopalasubramanian, Ganesh
Hi, Attached is a patch that implements * Prefetch with immediate offset in the range 0 to 32760 (multiple of 8). Added a predicate for this. * Prefetch with immediate offset - in the range -256 to 255 (Gets generated only when we have a negative offset. Generates prfum instruction).