On 28 May 2014, at 16:25 , Gopalasubramanian, Ganesh
wrote:
> Hi Philipp,
>
>> These changes look good to me.
>> We'll try them out on the benchmarks that caused us to add prefetching in
>> the first place.
>
> If you are OK, I would like to get these changes upstreamed.
Sorry for the delay
o:philipp.toms...@theobroma-systems.com]
Sent: Friday, February 28, 2014 2:58 PM
To: Gopalasubramanian, Ganesh
Cc: gcc-patches@gcc.gnu.org; pins...@gmail.com
Subject: Re: [AArch64 05/14] Add AArch64 'prefetch'-pattern.
Ganesh,
On 28 Feb 2014, at 10:13 , Gopalasubramanian, Ganesh
wrote:
> I
Ganesh,
On 28 Feb 2014, at 10:13 , Gopalasubramanian, Ganesh
wrote:
> I also have attached a patch that implements the following.
> * 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
Avoided top-posting and resending.
+ /* temporal locality */
+ return (INTVAL(operands[1])) ? \"prfm\\tPSTL1KEEP, [%0, #0]\" :
+\"prfm\\tPLDL1KEEP, [%0, #0]\"; }"
+ [(set_attr "type" "prefetch")]
+)
+
With the locality value received in the instruction pattern, I think it would
be safe to ha
With the locality value received in the instruction pattern, I think it would
be safe to handle them in prefetch instruction.
This helps especially AArch64 has prefetch instructions that can handle this
locality.
+(define_insn "prefetch"
+ [(prefetch (match_operand:DI 0 "address_operand" "r")
+
On Tue, Feb 18, 2014 at 1:09 PM, Philipp Tomsich
wrote:
Can you add a testcase or two for this?
Thanks,
Andrew
> ---
> gcc/config/aarch64/aarch64.md | 17 +
> gcc/config/arm/types.md | 2 ++
> 2 files changed, 19 insertions(+)
>
> diff --git a/gcc/config/aarch64/aarch64
