Fix UBSAN error, shifting 64 bit value by 64.

2020-03-13  Aaron Sawdey  <acsawdey@linux.ibm.com>

	PR target/92379
	* config/rs6000/rs6000.c (num_insns_constant_multi) Don't shift a
	64-bit value by 64 bits (UB).

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 4ea81e6c404bdaaa1322914543be47871d1cffac..2daa351088dca5e0a886b0a8eeb9296606b30c94 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2020-03-13  Aaron Sawdey  <acsawdey@linux.ibm.com>
+
+       PR target/92379
+       * config/rs6000/rs6000.c (num_insns_constant_multi) Don't shift a
+       64-bit value by 64 bits (UB).
+
 2020-03-13  Vladimir Makarov  <vmakarov@redhat.com>
 
        PR rtl-optimization/92303

diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index 24598aff6636a21e16fc21af0ac151590c3858e9..5798f924472ea8aed40e779698cfc3c764f77426 100644 (file)
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -5612,7 +5612,10 @@ num_insns_constant_multi (HOST_WIDE_INT value, machine_mode mode)
          && rs6000_is_valid_and_mask (GEN_INT (low), DImode))
        insns = 2;
       total += insns;
-      value >>= BITS_PER_WORD;
+      /* If BITS_PER_WORD is the number of bits in HOST_WIDE_INT, doing
+        it all at once would be UB. */
+      value >>= (BITS_PER_WORD - 1);
+      value >>= 1;
     }
   return total;
 }