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.


             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: 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).