public inbox for gcc-cvs@sourceware.org help / color / mirror / Atom feed
From: Michael Meissner <meissner@gcc.gnu.org> To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/users/meissner/heads/work095)] Update ChangeLog.meissner. Date: Fri, 22 Jul 2022 18:57:18 +0000 (GMT) [thread overview] Message-ID: <20220722185718.7B7693858405@sourceware.org> (raw) https://gcc.gnu.org/g:764ddb919d8ccdb6a01d9bbacaea15508f8d0187 commit 764ddb919d8ccdb6a01d9bbacaea15508f8d0187 Author: Michael Meissner <meissner@linux.ibm.com> Date: Fri Jul 22 14:57:00 2022 -0400 Update ChangeLog.meissner. 2022-07-22 Michael Meissner <meissner@linux.ibm.com> gcc/ * ChangeLog.meissner: Update. Diff: --- gcc/ChangeLog.meissner | 62 +++++++++++++++++++++----------------------------- 1 file changed, 26 insertions(+), 36 deletions(-) diff --git a/gcc/ChangeLog.meissner b/gcc/ChangeLog.meissner index 61d24a87eae..bd2990bce93 100644 --- a/gcc/ChangeLog.meissner +++ b/gcc/ChangeLog.meissner @@ -1,3 +1,26 @@ +==================== work095, patch #9. + +Do not enable -mblock-ops-vector-pair. + +Testing has shown that using the load vector pair and store vector pair +instructions for block moves has some performance issues on power10. + +A patch on June 11th modified the code so that GCC would not set +-mblock-ops-vector-pair by default if we are tuning for power10, but it +would set the option if we were tuning for a different machine and have +load and store vector pair instructions enabled. + +This patch just eliminates the code setting -mblock-ops-vector-pair at +all. + + +2022-07-22 Michael Meissner <meissner@linux.ibm.com> + +gcc/ + + * config/rs6000/rs6000.cc (rs6000_option_override_internal): + Do not enable -mblock-ops-vector-pair by default on power10. + ==================== work095, patch #8. Support IEEE 128-bit overload test data built-in functions. @@ -168,44 +191,11 @@ gcc/ attribute. (write_bif_static_init): Add support for ieeeld attribute. -==================== work095, patch #3. - -Make -mblock-ops-vector-pair option valid as an ISA option. - -I needed to turn off using load vector pair and store vector pair -instructions on the power10 for performance reasons. It is likely that in -future machines, we will want to enable these instructions for block moves -and copies. This patch adds the -mblock-ops-vector-pair option mask to -POWERPC_MASKS so that it is automatically set and cleared as we change the -target cpu. +==================== work095, patch #3 was reverted. -2022-07-19 Michael Meissner <meissner@linux.ibm.com> - -gcc/ - - * config/rs6000/rs6000-cpus.def (POWERPC_MASKS): Add - OPTION_MASK_BLOCK_OPS_VECTOR_PAIR. - * config/rs6000/rs6000.opt (-mblock-ops-vector-pair): Add a - comment. - -==================== work095, patch #2. - -Remove setting -mblock-ops-vector-pair on power10. - -Testing has shown that using the load vector pair and store vector pair -instructions for block moves has some performance issues on power10. This -patch does not set this option by default. If it is a win in other -machines, this flag can be set in the ISA options. - -2022-07-18 Michael Meissner <meissner@linux.ibm.com> - -gcc/ - - * config/rs6000/rs6000.cc (rs6000_option_override_internal): - Remove code setting -mblock-ops-vector-pair by default on - power10. +==================== work095, patch #2 was reverted. -==================== work095, patch #1 (reverted) +==================== work095, patch #1 was reverted. ==================== work095, branch setup.
next reply other threads:[~2022-07-22 18:57 UTC|newest] Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-07-22 18:57 Michael Meissner [this message] -- strict thread matches above, loose matches on Subject: below -- 2022-07-30 2:18 Michael Meissner 2022-07-30 1:34 Michael Meissner 2022-07-30 1:30 Michael Meissner 2022-07-29 19:30 Michael Meissner 2022-07-29 17:57 Michael Meissner 2022-07-29 5:58 Michael Meissner 2022-07-29 1:57 Michael Meissner 2022-07-29 1:47 Michael Meissner 2022-07-28 22:58 Michael Meissner 2022-07-28 22:31 Michael Meissner 2022-07-28 20:59 Michael Meissner 2022-07-23 0:12 Michael Meissner 2022-07-21 21:50 Michael Meissner 2022-07-18 19:41 Michael Meissner
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=20220722185718.7B7693858405@sourceware.org \ --to=meissner@gcc.gnu.org \ --cc=gcc-cvs@gcc.gnu.org \ /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: linkBe 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).