[PATCH] Fix PR 48143

Uros Bizjak ubizjak@gmail.com
Mon Mar 21 15:12:00 GMT 2011


2011/3/21 Andrey Belevantsev <abel@ispras.ru>:
> Hello,
>
> As noted in the PR audit trail, the scheduler crashes for this bug because
> we see an SSE insn without proper reservation.  Uros has pointed several
> such insns to me.  The following patch adding athlon_decode attributes is
> bootstrapped and tested on x86_64.
>
> Uros, you have preapproved the patch idea, but does it still look ok to you?
>  I have mostly copied reservations from bdver1 variants.
>
> Richi, Jakub, do you want this in 4.6.1?  4.6.0?
>
> Andrey
>
>
> 2011-03-21  Andrey Belevantsev  <abel@ispras.ru>
>
>        PR rtl-optimization/48143
>        * config/i386/sse.md (*sse2_cvtpd2dq): Add athlon_decode attribute.
>        (*sse2_cvttpd2dq, sse2_cvtss2sd, *sse2_cvtpd2ps,
>        sse2_cvtps2pd): Likewise.

Can you please cross-check these added attribute with the values in
Chapter C.7 (Table 18) and C.8 (Table 19) in [1] ? Otherwise, the
patch is OK.

[1] http://support.amd.com/us/Processor_TechDocs/25112.PDF

Thanks,
Uros.



More information about the Gcc-patches mailing list