[PATCH] add operand constraint for ix86 prefetch; fixes PR target/5379

Janis Johnson janis187@us.ibm.com
Mon Jan 21 16:24:00 GMT 2002


On Mon, Jan 21, 2002 at 04:15:28PM -0800, Richard Henderson wrote:
> On Mon, Jan 21, 2002 at 10:59:44AM -0800, Janis Johnson wrote:
> >  (define_expand "prefetch"
> > -  [(prefetch (match_operand:SI 0 "address_operand" "")
> > +  [(prefetch (match_operand:SI 0 "address_operand" "p")
> 
> Incorrect.

I saw the same problem with -mcpu=pentium3; how would I fix it for that?

> >  (define_insn "*prefetch_sse"
> > -  [(prefetch (match_operand:SI 0 "address_operand" "")
> > +  [(prefetch (match_operand:SI 0 "address_operand" "p")
> 
> Correct.
> 
> > +/* { dg-do compile { target i?86-*-* } } */
> > +/* { dg-options "-O2 -fprefetch-loop-arrays -march=athlon" } */
> 
> Prefer that you use
> 
> /* { dg-do compile } */
> /* { dg-options "-O2 -fprefetch-loop-arrays" } */

Should I add -w for this one so that there won't be warnings on targets
that don't support prefetch?

> /* { dg-options "-O2 -fprefetch-loop-arrays -march=athlon" { target i?86-*-* } } */
> 
> 
> r~



More information about the Gcc-patches mailing list