Add some gcc_asserts for vector extract processing.
authorMichael Meissner <meissner@linux.ibm.com>
Mon, 3 Feb 2020 20:50:39 +0000 (15:50 -0500)
committerMichael Meissner <meissner@linux.ibm.com>
Mon, 3 Feb 2020 20:50:39 +0000 (15:50 -0500)
2020-02-03  Michael Meissner  <meissner@linux.ibm.com>

* config/rs6000/rs6000.c (rs6000_adjust_vec_address): Add some
gcc_asserts.

gcc/ChangeLog
gcc/config/rs6000/rs6000.c

index 4ded82cd6fd5117b8df3c2add52c1d7dc2b9c1dc..b66cec2b85f2439d97a96351a1dd35f0952847d4 100644 (file)
@@ -1,3 +1,8 @@
+2020-02-03  Michael Meissner  <meissner@linux.ibm.com>
+
+       * config/rs6000/rs6000.c (rs6000_adjust_vec_address): Add some
+       gcc_asserts.
+
 2020-02-03  Segher Boessenkool  <segher@kernel.crashing.org>
 
        * config/rs6000/constraints.md: Improve documentation.
index fc36bb6714b2a4c922e903e2ebe333c6bdaeefcd..8f72350f331140ffc14c8d65ae0eb0f03454a615 100644 (file)
@@ -6768,6 +6768,9 @@ rs6000_adjust_vec_address (rtx scalar_reg,
   rtx new_addr;
   bool valid_addr_p;
 
+  gcc_assert (!reg_mentioned_p (base_tmp, addr));
+  gcc_assert (!reg_mentioned_p (base_tmp, element));
+
   /* Vector addresses should not have PRE_INC, PRE_DEC, or PRE_MODIFY.  */
   gcc_assert (GET_RTX_CLASS (GET_CODE (addr)) != RTX_AUTOINC);
 
@@ -6777,6 +6780,10 @@ rs6000_adjust_vec_address (rtx scalar_reg,
     element_offset = GEN_INT (INTVAL (element) * scalar_size);
   else
     {
+      /* All insns should use the 'Q' constraint (address is a single register)
+        if the element number is not a constant.  */
+      gcc_assert (REG_P (addr) || SUBREG_P (addr));
+
       int byte_shift = exact_log2 (scalar_size);
       gcc_assert (byte_shift >= 0);