From: Evandro Menezes <e.menezes@samsung.com>
Date: Wed, 4 May 2016 20:47:25 +0000 (+0000)
Subject: [AArch64] Add cheap as move instructions for Exynos M1
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=00ed93ab9217ef49140249398781cb2129b4dff2;p=llvm

[AArch64] Add cheap as move instructions for Exynos M1

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

diff --git a/lib/Target/AArch64/AArch64InstrInfo.cpp b/lib/Target/AArch64/AArch64InstrInfo.cpp
index 46d90abbe2a..b09cfc0ff55 100644
--- a/lib/Target/AArch64/AArch64InstrInfo.cpp
+++ b/lib/Target/AArch64/AArch64InstrInfo.cpp
@@ -545,9 +545,11 @@ static bool canBeExpandedToORR(const MachineInstr *MI, unsigned BitSize) {
 // micro-architecture target hook should be introduced here in future.
 bool AArch64InstrInfo::isAsCheapAsAMove(const MachineInstr *MI) const {
   if (!Subtarget.isCortexA57() && !Subtarget.isCortexA53() &&
-      !Subtarget.isKryo())
+      !Subtarget.isExynosM1() && !Subtarget.isKryo())
     return MI->isAsCheapAsAMove();
 
+  unsigned Imm;
+
   switch (MI->getOpcode()) {
   default:
     return false;
@@ -557,7 +559,17 @@ bool AArch64InstrInfo::isAsCheapAsAMove(const MachineInstr *MI) const {
   case AArch64::ADDXri:
   case AArch64::SUBWri:
   case AArch64::SUBXri:
-    return (MI->getOperand(3).getImm() == 0);
+    return (Subtarget.isExynosM1() ||
+            MI->getOperand(3).getImm() == 0);
+
+  // add/sub on register with shift
+  case AArch64::ADDWrs:
+  case AArch64::ADDXrs:
+  case AArch64::SUBWrs:
+  case AArch64::SUBXrs:
+    Imm = MI->getOperand(3).getImm();
+    return (Subtarget.isExynosM1() &&
+            AArch64_AM::getArithShiftValue(Imm) < 4);
 
   // logical ops on immediate
   case AArch64::ANDWri:
@@ -582,6 +594,25 @@ bool AArch64InstrInfo::isAsCheapAsAMove(const MachineInstr *MI) const {
   case AArch64::ORRWrr:
   case AArch64::ORRXrr:
     return true;
+
+  // logical ops on register with shift
+  case AArch64::ANDWrs:
+  case AArch64::ANDXrs:
+  case AArch64::BICWrs:
+  case AArch64::BICXrs:
+  case AArch64::EONWrs:
+  case AArch64::EONXrs:
+  case AArch64::EORWrs:
+  case AArch64::EORXrs:
+  case AArch64::ORNWrs:
+  case AArch64::ORNXrs:
+  case AArch64::ORRWrs:
+  case AArch64::ORRXrs:
+    Imm = MI->getOperand(3).getImm();
+    return (Subtarget.isExynosM1() &&
+            AArch64_AM::getShiftValue(Imm) < 4 &&
+            AArch64_AM::getShiftType(Imm) == AArch64_AM::LSL);
+
   // If MOVi32imm or MOVi64imm can be expanded into ORRWri or
   // ORRXri, it is as cheap as MOV
   case AArch64::MOVi32imm: