Ganesh,

On 28 Feb 2014, at 10:13 , Gopalasubramanian, Ganesh 
<ganesh.gopalasubraman...@amd.com> 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 -256 to 255 (Gets 
> generated only when we have a negative offset. Generates prfum instruction). 
> Added a predicate for this.
> *     Prefetch with register offset. (modified for printing the locality)

These changes look good to me.
We’ll try them out on the benchmarks that caused us to add prefetching in the 
first place.

Best,
Philipp.

Reply via email to