From: Uros Bizjak <ubizjak@gmail.com>
To: Vladimir Yakovlev <vbyakovl23@gmail.com>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] Vzeroupper placement/47440
Date: Mon, 05 Nov 2012 19:52:00 -0000 [thread overview]
Message-ID: <CAFULd4aJyjzEAQRwtr+QudJYXdt3pSxWEsFTxf8AOiNbDKjdqg@mail.gmail.com> (raw)
Hello!
> 2012-11-04 Vladimir Yakovlev <vladimir.b.yakovlev@intel.com>
>
> * mode-switching.c (create_pre_exit): Added code for maybe_builtin_apply case.
The part above is already committed.
> * config/i386/i386-protos.h (emit_i387_cw_initialization): Deleted.
> (emit_vzero): Added prototype.
> (ix86_mode_entry): Likewise.
> (ix86_mode_exit): Likewise.
> (ix86_emit_mode_set): Likewise.
>
> * config/i386/i386.c (typedef struct block_info_def): Deleted.
> (define BLOCK_INFO): Deleted.
> (check_avx256_stores): Added checking for MEM_P.
> (move_or_delete_vzeroupper_2): Deleted.
> (move_or_delete_vzeroupper_1): Deleted.
> (move_or_delete_vzeroupper): Deleted.
> (ix86_maybe_emit_epilogue_vzeroupper): Deleted.
> (function_pass_avx256_p): Deleted.
> (ix86_function_ok_for_sibcall): Deleted disabling sibcall.
> (nit_cumulative_args): Deleted initialization of of avx256 fields of
> cfun->machine.
> (ix86_emit_restore_sse_regs_using_mov): Deleted vzeroupper generation.
> (ix86_expand_epilogue): Likewise.
> (ix86_avx_u128_mode_needed): New.
> (ix86_i387_mode_needed): Renamed ix86_mode_needed.
> (ix86_mode_needed): New.
> (ix86_avx_u128_mode_after): New.
> (ix86_mode_after): New.
> (ix86_avx_u128_mode_entry): New.
> (ix86_mode_entry): New.
> (ix86_avx_u128_mode_exit): New.
> (ix86_mode_exit): New.
> (ix86_emit_mode_set): New.
> (ix86_expand_call): Deleted vzeroupper generation.
> (ix86_split_call_vzeroupper): Deleted.
> (ix86_init_machine_status): Initialzed optimize_mode_switching.
> (ix86_expand_special_args_builtin): Changed.
> (ix86_reorg): Deleted a call of move_or_delete_vzeroupper.
>
> * config/i386/i386.h (VALID_AVX256_REG_OR_OI_MODE): New.
> (AVX_U128): New.
> (avx_u128_state): New.
> (NUM_MODES_FOR_MODE_SWITCHING): Added AVX_U128_ANY.
> (MODE_AFTER): New.
> (MODE_ENTRY): New.
> (MODE_EXIT): New.
> (EMIT_MODE_SET): Changed.
> (machine_function): Deleted avx256 fields.
>
> * config/i386/i386.md (UNSPEC_CALL_NEEDS_VZEROUPPER): Deleted.
> (define_insn_and_split "*call_vzeroupper"): Deleted.
> (define_insn_and_split "*call_rex64_ms_sysv_vzeroupper"): Deleted.
> (define_insn_and_split "*sibcall_vzeroupper"): Deleted.
> (define_insn_and_split "*call_pop_vzeroupper"): Deleted.
> (define_insn_and_split "*sibcall_pop_vzeroupper"): Deleted.
> (define_insn_and_split "*call_value_vzeroupper"): Deleted.
> (define_insn_and_split "*sibcall_value_vzeroupper"): Deleted.
> (define_insn_and_split "*call_value_rex64_ms_sysv_vzeroupper"): Deleted.
> (define_insn_and_split "*call_value_pop_vzeroupper"): Deleted.
> (define_insn_and_split "*sibcall_value_pop_vzeroupper"): Deleted.
> (define_expand "return"): Deleted vzeroupper emitting.
> (define_expand "simple_return"): Deleted.
>
> * config/i386/predicates.md (vzeroupper_operation): New.
>
> * config/i386/sse.md (avx_vzeroupper): Changed.
>
> 2012-11-04 Vladimir Yakovlev <vladimir.b.yakovlev@intel.com>
>
> * gcc.target/i386/avx-vzeroupper-5.c: Changed scan-assembler-times.
> gcc.target/i386/avx-vzeroupper-8.c: Likewise.
> gcc.target/i386/avx-vzeroupper-9.c: Likewise.
> gcc.target/i386/avx-vzeroupper-10.c: Likewise.
> gcc.target/i386/avx-vzeroupper-11.c: Likewise.
> gcc.target/i386/avx-vzeroupper-12.c: Likewise.
> gcc.target/i386/avx-vzeroupper-19.c: Likewis.
> gcc.target/i386/avx-vzeroupper-27.c: New.
OK for mainline SVN, please commit.
Thanks,
Uros.
next reply other threads:[~2012-11-05 19:52 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-05 19:52 Uros Bizjak [this message]
2012-11-06 10:30 ` Kirill Yukhin
2012-11-06 22:11 ` H.J. Lu
2012-11-06 22:18 ` Jakub Jelinek
2012-11-07 7:03 ` Uros Bizjak
2012-11-07 7:08 ` Uros Bizjak
2012-11-07 8:05 ` Jakub Jelinek
2012-11-07 9:42 ` Uros Bizjak
2012-11-07 12:19 ` Uros Bizjak
2012-11-07 15:09 ` Vladimir Yakovlev
2012-11-08 5:49 ` Vladimir Yakovlev
[not found] <CAK1BsWrsVu4TRW50RW0X7G4RSguSAjhqFPe-tkeXKaurr=sX1A@mail.gmail.com>
[not found] ` <CAFULd4b0y6GGZsn1s4-RXc1mAvZGrhGd4YQBhfLgeMWmv2eXPA@mail.gmail.com>
[not found] ` <CAK1BsWoL5hsfZprf-a8zxG+Bhe9SwGFwqxHxOw9UX+bbsFD5oQ@mail.gmail.com>
[not found] ` <CAFULd4bJXT-nnAk6HCn2C=+jhfiUD-fAe3LK8AYd9jgqQQHvKQ@mail.gmail.com>
[not found] ` <CAFULd4bdxuKbYYS7TcyRfjNukLvJ0d5pOD7zJGAyKEQLPq7z2Q@mail.gmail.com>
[not found] ` <CAK1BsWpL69eRHTD8dzVOm9xtOqtjcr6z3B2tvb_VikWPzKT0Dw@mail.gmail.com>
[not found] ` <CAFULd4YaVLCYF=Huw_kDozTBTcZnGUAy7xOcV+VEweOWZ5Cigg@mail.gmail.com>
[not found] ` <CAFULd4YyRVY4BzD+csZAqCCmB7v3YEwAaOpNW9QsMXEbCkFw+Q@mail.gmail.com>
2012-11-09 12:18 ` [off-list] " Vladimir Yakovlev
2012-11-09 12:29 ` Uros Bizjak
2012-11-09 12:36 ` Jakub Jelinek
2012-11-09 12:48 ` Uros Bizjak
2012-11-09 13:28 ` Uros Bizjak
2012-11-16 7:50 ` Uros Bizjak
-- strict thread matches above, loose matches on Subject: below --
2012-11-04 13:29 Uros Bizjak
2012-11-04 17:59 ` Uros Bizjak
[not found] ` <CAK1BsWpoD4AVB_4+J6snJgs4BF1Jbiw-RrifvZiiAm21qRURew@mail.gmail.com>
[not found] ` <CAFULd4Y5zDhMH3h34Lt0O5xNG+xibDJih7q2_ctef7nqSNJcOQ@mail.gmail.com>
2012-11-04 20:28 ` Vladimir Yakovlev
2012-11-04 4:31 Vladimir Yakovlev
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFULd4aJyjzEAQRwtr+QudJYXdt3pSxWEsFTxf8AOiNbDKjdqg@mail.gmail.com \
--to=ubizjak@gmail.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=vbyakovl23@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).