rs6000: Make constant formation a tiny bit better
authorSegher Boessenkool <segher@kernel.crashing.org>
Fri, 19 May 2017 11:29:21 +0000 (13:29 +0200)
committerSegher Boessenkool <segher@gcc.gnu.org>
Fri, 19 May 2017 11:29:21 +0000 (13:29 +0200)
If we cannot load a constant into a register in one insn, and that
constant is a valid mask (for rotate instructions), we currently
prefer to load -1 and then mask it.  This patch makes us not do that if
instead we could use two add or or instructions, since those are
sometimes faster on certain CPUs (and never are slower).

* config/rs6000/rs6000.md (splitter to load of -1 and mask): Don't
use this splitter if two add or or instructions would also work for
the constant we want to generate.

From-SVN: r248265

gcc/ChangeLog
gcc/config/rs6000/rs6000.md

index 9e386f7..9006383 100644 (file)
@@ -1,3 +1,9 @@
+2017-05-19  Segher Boessenkool  <segher@kernel.crashing.org>
+
+       * config/rs6000/rs6000.md (splitter to load of -1 and mask): Don't
+       use this splitter if two add or or instructions would also work for
+       the constant we want to generate.
+
 2017-05-19  Richard Biener  <rguenther@suse.de>
 
        PR build/80821
index 26baaaf..799d786 100644 (file)
        (match_operand:DI 1 "const_int_operand"))]
   "TARGET_POWERPC64
    && num_insns_constant (operands[1], DImode) > 1
+   && !IN_RANGE (INTVAL (operands[1]), -0x80000000, 0xffffffff)
    && rs6000_is_valid_and_mask (operands[1], DImode)"
   [(set (match_dup 0)
        (const_int -1))