commit cc69fa29cf14a3c4340f1ea4a32a1d24772b605d
author Eric Holk <eholk@chromium.org> Wed Feb 10 13:07:06 2016 -0800
committer Eric Holk <eholk@chromium.org> Wed Feb 10 13:07:06 2016 -0800
tree 2b05060dc39f47b018d527dd24aa429ba74e8438
parent 8aa396610b7baf728631a43ea16ad3d13e38397a

ARM32 vector ops - scalarize icmp, fcmp and cast.

This is part of a sequence of patches to quickly fill out vector
support by scalarizing the remaining operations. Later we can work to
generate better code.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4076
R=stichnot@chromium.org

Review URL: https://codereview.chromium.org/1683153003 .

src/IceTargetLoweringARM32.cpp

diff --git a/src/IceTargetLoweringARM32.cpp b/src/IceTargetLoweringARM32.cpp
index 2da5542..2572f7c 100644
--- a/src/IceTargetLoweringARM32.cpp
+++ b/src/IceTargetLoweringARM32.cpp
@@ -577,8 +577,18 @@
     Variable *Dest = Instr->getDest();
     Operand *Src0 = Instr->getSrc(0);
     const Type DestTy = Dest->getType();
-    const InstCast::OpKind CastKind =
-        llvm::cast<InstCast>(Instr)->getCastKind();
+    auto *CastInstr = llvm::cast<InstCast>(Instr);
+    const InstCast::OpKind CastKind = CastInstr->getCastKind();
+
+    if (isVectorType(DestTy)) {
+      scalarizeUnaryInstruction(
+          Dest, Src0, [this, CastKind](Variable *Dest, Variable *Src) {
+            return Context.insert<InstCast>(CastKind, Dest, Src);
+          });
+      CastInstr->setDeleted();
+      return;
+    }
+
     switch (CastKind) {
     default:
       return;
@@ -723,6 +733,36 @@
     }
     llvm::report_fatal_error("Control flow should never have reached here.");
   }
+  case Inst::Icmp: {
+    Variable *Dest = Instr->getDest();
+    const Type DestTy = Dest->getType();
+    if (isVectorType(DestTy)) {
+      auto *CmpInstr = llvm::cast<InstIcmp>(Instr);
+      const auto Condition = CmpInstr->getCondition();
+      scalarizeInstruction(
+          Dest, CmpInstr->getSrc(0), CmpInstr->getSrc(1),
+          [this, Condition](Variable *Dest, Variable *Src0, Variable *Src1) {
+            return Context.insert<InstIcmp>(Condition, Dest, Src0, Src1);
+          });
+      CmpInstr->setDeleted();
+    }
+    return;
+  }
+  case Inst::Fcmp: {
+    Variable *Dest = Instr->getDest();
+    const Type DestTy = Dest->getType();
+    if (isVectorType(DestTy)) {
+      auto *CmpInstr = llvm::cast<InstFcmp>(Instr);
+      const auto Condition = CmpInstr->getCondition();
+      scalarizeInstruction(
+          Dest, CmpInstr->getSrc(0), CmpInstr->getSrc(1),
+          [this, Condition](Variable *Dest, Variable *Src0, Variable *Src1) {
+            return Context.insert<InstFcmp>(Condition, Dest, Src0, Src1);
+          });
+      CmpInstr->setDeleted();
+    }
+    return;
+  }
   }
 }
 
@@ -4194,9 +4234,6 @@
 }
 
 TargetARM32::CondWhenTrue TargetARM32::lowerIcmpCond(const InstIcmp *Instr) {
-  assert(Instr->getSrc(0)->getType() != IceType_i1);
-  assert(Instr->getSrc(1)->getType() != IceType_i1);
-
   Operand *Src0 = legalizeUndef(Instr->getSrc(0));
   Operand *Src1 = legalizeUndef(Instr->getSrc(1));
 
@@ -4233,6 +4270,7 @@
   switch (Src0->getType()) {
   default:
     llvm::report_fatal_error("Unhandled type in lowerIcmpCond");
+  case IceType_i1:
   case IceType_i8:
   case IceType_i16:
     return lowerInt8AndInt16IcmpCond(Condition, Src0, Src1);