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/work144-vsize)] Update ChangeLog.*
Date: Wed, 15 Nov 2023 18:35:00 +0000 (GMT)	[thread overview]
Message-ID: <20231115183500.8F8C93858D20@sourceware.org> (raw)

https://gcc.gnu.org/g:107a6e004e5fac7caa42aea659f1f8f8046e5ea2

commit 107a6e004e5fac7caa42aea659f1f8f8046e5ea2
Author: Michael Meissner <meissner@linux.ibm.com>
Date:   Wed Nov 15 13:34:57 2023 -0500

    Update ChangeLog.*

Diff:
---
 gcc/ChangeLog.vsize | 163 +++++++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 162 insertions(+), 1 deletion(-)

diff --git a/gcc/ChangeLog.vsize b/gcc/ChangeLog.vsize
index a1aeb7b3527..eff3cb95398 100644
--- a/gcc/ChangeLog.vsize
+++ b/gcc/ChangeLog.vsize
@@ -1,3 +1,164 @@
+==================== Branch work144-vsize, patch #401 ====================
+
+Add init, extract, and set support for vector pair modes.
+
+2023-11-15  Michael Meissner  <meissner@linux.ibm.com>
+
+gcc/
+
+	* config/rs6000/vector.md (VEC_E): Add vector pair modes.
+	(VEC_base_l): Likewise.
+
+==================== Branch work144-vsize, patch #400 ====================
+
+Add support for -mvector-size-32.
+
+2023-11-09  Michael Meissner  <meissner@linux.ibm.com>
+
+gcc/
+
+	* config/rs6000/predicates.md (const_0_to_31_operand): New predicate.
+	* config/rs6000/rs6000-c.cc (rs6000_cpu_cpp_builtins): Define
+	__VECTOR_SIZE_32__ if -mvector-size-32 was used.
+	* config/rs6000/rs6000-cpus.def (OTHER_POWER10_MASKS): Add
+	-mvector-size-32.
+	(POWERPC_MASKS): Likewise.
+	* config/rs6000/rs6000-protos.h (vector_pair_to_vector_mode): New
+	declaration.
+	(rs6000_adjust_for_vector_pair): Likewise.
+	(split_unary_vector_pair): Likewise.
+	(split_binary_vector_pair): Likewise.
+	(split_fma_vector_pair): Likewise.
+	* config/rs6000/rs6000.cc (rs6000_hard_regno_mode_ok_uncached): Add
+	support for 32-byte vector types created with -mvector-size-32.
+	(rs6000_modes_tieable_p): Make all 32-byte vectors tie with other
+	32-byte vectors.
+	(rs6000_debug_reg_global): If -mdebug=reg, print whether
+	-mvector-size-32 was enabled.
+	(rs6000_init_hard_regno_mode_ok): Add support for 32-byte vectors.
+	(rs6000_option_override_internal): Add checking for -mvector-size-32.
+	(rs6000_opt_masks): Add -mvector-size-32.
+	(rs6000_expand_vector_extract): Add support for 32-byte vectors.
+	(reg_offset_addressing_ok_p): Likewise.
+	(rs6000_emit_move): Likewise.
+	(rs6000_preferred_reload_class): Likewise.
+	(vector_pair_to_vector_mode): New vector pair helper function.
+	(rs6000_adjust_for_vector_pair): Likewise.
+	(rs6000_split_vpair_constan): Likewise.
+	(split_unary_vector_pair): Likewise.
+	(split_binary_vector_pair): Likewise.
+	(split_fma_vector_pair): Likewise.
+	(rs6000_split_multireg_move): Add support for 32-byte vectors.
+	* config/rs6000/rs6000.h (VECTOR_PAIR_MODE): New macro.
+	* config/rs6000/rs6000.md (wd attribute): Add 32-byte vector modes.
+	(RELOAD): Likewise.
+	(toplevel): Include vector-pair.md.
+	* config/rs6000/rs6000.opt (-mvector-size-32): New option.
+	* config/rs6000/t-rs6000 (MD_INCLUDES): Add vector-pair.md.
+	* config/rs6000/vector-pair.md: New file.
+	* config/rs6000/vector.md (VEC_base): Add 32-byte vector modes.
+	* config/rs6000/vsx.md (VSX_EXTRACT_PREDICATE): Likewise.
+	(VSX_EX): Likewise.
+	(VPAIR_V4DI_V4DF): New mode iterator.
+	(VPAIR_VECTOR): New mode attribute.
+	(vpair_vector): Likewise.
+	(vsx_extract_<mode>, VPAIR_V4DF_V4DI iterator): New extract insn for
+	vector pair support.
+	(vsx_extract_v8sf): Likewise.
+	(vsx_extract_<mode>, VPAIR_SMALL_INT iterator): Likewise.
+
+gcc/testsuite/
+
+	* gcc.target/powerpc/vector-size-32-1.c: New test.
+	* gcc.target/powerpc/vector-size-32-2.c: New test.
+	* gcc.target/powerpc/vector-size-32-3.c: New test.
+	* gcc.target/powerpc/vector-size-32-4.c: New test.
+	* gcc.target/powerpc/vector-size-32-5.c: New test.
+	* gcc.target/powerpc/vector-size-32-6.c: New test.
+
+
 ==================== Branch work144-vsize, patch #1 (from work144) ====================
 
-See ChangeLog.meissner
+Power10: Add options to disable load and store vector pair.
+
+This is version 2 of the patch to add -mno-load-vector-pair and
+-mno-store-vector-pair undocumented tuning switches.
+
+The differences between the first version of the patch and this version is that
+I added explicit RTL abi attributes for when the compiler can generate the load
+vector pair and store vector pair instructions.  By having this attribute, the
+movoo insn has separate alternatives for when we generate the instruction and
+when we want to split the instruction into 2 separate vector loads or stores.
+
+In the first version of the patch, I had previously provided built-in functions
+that would always generate load vector pair and store vector pair instructions
+even if these instructions are normally disabled.  I found these built-ins
+weren't specified like the other vector pair built-ins, and I didn't include
+documentation for the built-in functions.  If we want such built-in functions,
+we can add them as a separate patch later.
+
+In addition, since both versions of the patch adds #pragma target and attribute
+support to change the results for individual functions, we can select on a
+function by function basis what the defaults for load/store vector pair is.
+
+The original text for the patch is:
+
+In working on some future patches that involve utilizing vector pair
+instructions, I wanted to be able to tune my program to enable or disable using
+the vector pair load or store operations while still keeping the other
+operations on the vector pair.
+
+This patch adds two undocumented tuning options.  The -mno-load-vector-pair
+option would tell GCC to generate two load vector instructions instead of a
+single load vector pair.  The -mno-store-vector-pair option would tell GCC to
+generate two store vector instructions instead of a single store vector pair.
+
+If either -mno-load-vector-pair is used, GCC will not generate the indexed
+stxvpx instruction.  Similarly if -mno-store-vector-pair is used, GCC will not
+generate the indexed lxvpx instruction.  The reason for this is to enable
+splitting the {,p}lxvp or {,p}stxvp instructions after reload without needing a
+scratch GPR register.
+
+The default for -mcpu=power10 is that both load vector pair and store vector
+pair are enabled.
+
+I added code so that the user code can modify these settings using either a
+'#pragma GCC target' directive or used __attribute__((__target__(...))) in the
+function declaration.
+
+I added tests for the switches, #pragma, and attribute options.
+
+I have built this on both little endian power10 systems and big endian power9
+systems doing the normal bootstrap and test.  There were no regressions in any
+of the tests, and the new tests passed.  Can I check this patch into the master
+branch?
+
+2023-11-09  Michael Meissner  <meissner@linux.ibm.com>
+
+gcc/
+
+	* config/rs6000/mma.md (movoo): Add support for -mno-load-vector-pair and
+	-mno-store-vector-pair.
+	* config/rs6000/rs6000-cpus.def (OTHER_POWER10_MASKS): Add support for
+	-mload-vector-pair and -mstore-vector-pair.
+	(POWERPC_MASKS): Likewise.
+	* config/rs6000/rs6000.cc (rs6000_setup_reg_addr_masks): Only allow
+	indexed mode for OOmode if we are generating both load vector pair and
+	store vector pair instructions.
+	(rs6000_option_override_internal): Add support for -mno-load-vector-pair
+	and -mno-store-vector-pair.
+	(rs6000_opt_masks): Likewise.
+	* config/rs6000/rs6000.md (isa attribute): Add lxvp and stxvp
+	attributes.
+	(enabled attribute): Likewise.
+	* config/rs6000/rs6000.opt (-mload-vector-pair): New option.
+	(-mstore-vector-pair): Likewise.
+
+gcc/testsuite/
+
+	* gcc.target/powerpc/vector-pair-attribute.c: New test.
+	* gcc.target/powerpc/vector-pair-pragma.c: New test.
+	* gcc.target/powerpc/vector-pair-switch1.c: New test.
+	* gcc.target/powerpc/vector-pair-switch2.c: New test.
+	* gcc.target/powerpc/vector-pair-switch3.c: New test.
+	* gcc.target/powerpc/vector-pair-switch4.c: New test.

             reply	other threads:[~2023-11-15 18:35 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-15 18:35 Michael Meissner [this message]
2023-11-15 19:35 Michael Meissner
2023-11-15 19:58 Michael Meissner
2023-11-16  0:43 Michael Meissner
2023-11-16  3:15 Michael Meissner
2023-11-16  7:17 Michael Meissner
2023-11-16 17:47 Michael Meissner
2023-11-16 20:16 Michael Meissner
2023-11-16 22:02 Michael Meissner
2023-11-16 22:23 Michael Meissner
2023-11-16 22:38 Michael Meissner
2023-11-17  2:55 Michael Meissner
2023-11-17  3:46 Michael Meissner
2023-11-17 16:23 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=20231115183500.8F8C93858D20@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).