[PATCH] Increase rtx_cost of sse_to_integer in skylake_cost.

[Jeff Law]

On 9/15/20 9:20 PM, Hongtao Liu via Gcc-patches wrote:
> Hi:
>   Rtx cost of sse_to_integer would be used by pass_stv as a
> measurement for the scalar-to-vector transformation. As
> https://gcc.gnu.org/pipermail/gcc-patches/2019-August/528839.html
> indicates, movement between sse regs and gprs should be much expensive
> than movement inside gprs(which is 2 as default). This patch would
> also fix "pr96861".
>
>   Bootstrap is ok, regression test is ok for both "i386.exp=*
> --target_board='unix{-m32,}'" and "i386.exp=*
> --target_board='unix{-m32\ -march=cascadelake,-m64\
> -march=cascadelake}"".
>   No big impact on SPEC2017.
>   Ok for trunk?
>
> gcc/ChangeLog
>
>         PR target/96861
>         * config/i386/x86-tune-costs.h (skylake_cost): increase rtx
>         cost of sse_to_integer from 2 to 6.
>
> gcc/testsuite
>
>         * gcc.target/i386/pr95021-3.c: Add -mtune=generic.

I'll defer to HJ's judgement here.  If he's OK with it, then it's fine
by me.


jeff


-------------- next part --------------
A non-text attachment was scrubbed...
Name: pEpkey.asc
Type: application/pgp-keys
Size: 1733 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc-patches/attachments/20200916/a7da5700/attachment.bin>