From: Nate Begeman <natebegeman@mac.com>
Date: Fri, 15 Oct 2004 00:50:19 +0000 (+0000)
Subject: Better codegen of binary integer ops with 32 bit immediate operands.
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=e0c83a86b0f609e7212fba6679453b8e5bd7fa33;p=oota-llvm.git

Better codegen of binary integer ops with 32 bit immediate operands.
This transformation fires a few dozen times across the testsuite.

For example, int test2(int X) { return X ^ 0x0FF00FF0; }
Old:
_test2:
        lis r2, 4080
        ori r2, r2, 4080
        xor r3, r3, r2
        blr

New:
_test2:
        xoris r3, r3, 4080
        xori r3, r3, 4080
        blr


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@17004 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Target/PowerPC/PPC32ISelSimple.cpp b/lib/Target/PowerPC/PPC32ISelSimple.cpp
index 890ccb95dc6..fd5f671ac4e 100644
--- a/lib/Target/PowerPC/PPC32ISelSimple.cpp
+++ b/lib/Target/PowerPC/PPC32ISelSimple.cpp
@@ -32,7 +32,7 @@
 using namespace llvm;
 
 namespace {
-  Statistic<> NumClear("ppc-codegen", "Number of AND turned into mask");
+  Statistic<> NumHiAndLo("ppc-codegen", "Number of 32b imms not loaded");
 
   /// TypeClass - Used by the PowerPC backend to group LLVM types by their basic
   /// PPC Representation.
@@ -2128,13 +2128,19 @@ void PPC32ISel::emitBinaryConstOperation(MachineBasicBlock *MBB,
   if (Opcode == 2) {
     unsigned MB, ME, mask = CI->getRawValue();
     if (isRunOfOnes(mask, MB, ME)) {
-      ++NumClear;
       BuildMI(*MBB, IP, PPC::RLWINM, 4, DestReg).addReg(Op0Reg).addImm(0)
         .addImm(MB).addImm(ME);
       return;
     }
   }
 
+  // PowerPC 16 bit signed immediates are sign extended before use by the
+  // instruction.  Therefore, we can only split up an add of a reg with a 32 bit
+  // immediate into addis and addi if the sign bit of the low 16 bits is cleared
+  // so that for register A, const imm X, we don't end up with
+  // A + XXXX0000 + FFFFXXXX.
+  bool WontSignExtend = (0 == (Op1->getRawValue() & 0x8000));
+
   // For Add, Sub, and SubF the instruction takes a signed immediate.  For And,
   // Or, and Xor, the instruction takes an unsigned immediate.  There is no 
   // shifted immediate form of SubF so disallow its opcode for those constants.
@@ -2152,6 +2158,20 @@ void PPC32ISel::emitBinaryConstOperation(MachineBasicBlock *MBB,
     else
       BuildMI(*MBB, IP, ImmOpTab[1][Opcode], 2, DestReg).addReg(Op0Reg)
         .addZImm(Op1->getRawValue() >> 16);
+  } else if ((Opcode < 2 && WontSignExtend) || Opcode == 3 || Opcode == 4) {
+    unsigned TmpReg = makeAnotherReg(Op1->getType());
+    ++NumHiAndLo;
+    if (Opcode < 2) {
+      BuildMI(*MBB, IP, ImmOpTab[1][Opcode], 2, TmpReg).addReg(Op0Reg)
+        .addSImm(Op1->getRawValue() >> 16);
+      BuildMI(*MBB, IP, ImmOpTab[0][Opcode], 2, DestReg).addReg(TmpReg)
+        .addSImm(Op1->getRawValue());
+    } else {
+      BuildMI(*MBB, IP, ImmOpTab[1][Opcode], 2, TmpReg).addReg(Op0Reg)
+        .addZImm(Op1->getRawValue() >> 16);
+      BuildMI(*MBB, IP, ImmOpTab[0][Opcode], 2, DestReg).addReg(TmpReg)
+        .addZImm(Op1->getRawValue());
+    }
   } else {
     unsigned Op1Reg = getReg(Op1, MBB, IP);
     BuildMI(*MBB, IP, OpTab[Opcode], 2, DestReg).addReg(Op0Reg).addReg(Op1Reg);