Add vmov between floating point registers to ARM assembler.

src/IceInstARM32.cpp

 //===- subzero/src/IceInstARM32.cpp - ARM32 instruction implementation ----===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
@@ skipping 1042 matching lines @@
 void InstARM32Mov::emitMultiDestSingleSource(const Cfg *Func) const {
   if (!BuildDefs::dump())
     return;
   Ostream &Str = Func->getContext()->getStrEmit();
   Variable *DestLo = getDest();
   Variable *DestHi = getDestHi();
   auto *Src = llvm::cast<Variable>(getSrc(0));
 
   assert(DestHi->hasReg());
   assert(DestLo->hasReg());
   assert(llvm::isa<Variable>(Src) && Src->hasReg());

[Jim Stichnoth, 2016/01/27 22:10:49]

llvm::isa<Variable>(Src) is redundant with the llvm::cast<Variable> above and
can be removed.

[Karl, 2016/01/27 22:58:09]

Done.

 
   Str << "\t"
          "vmov" << getPredicate() << "\t";
   DestLo->emit(Func);
   Str << ", ";
   DestHi->emit(Func);
   Str << ", ";
   Src->emit(Func);
 }
 
@@ skipping 77 matching lines @@
   Src0->emit(Func);
 }
 
 void InstARM32Mov::emitIASScalarVFPMove(const Cfg *Func) const {
   auto *Asm = Func->getAssembler<ARM32::AssemblerARM32>();
   Operand *Src0 = getSrc(0);
   Variable *Dest = getDest();
   switch (Dest->getType()) {
   default:
     assert(false && "Do not know how to emit scalar FP move for type.");
-    return;
+    break;
   case IceType_f32:
-    if (const auto *FpImm = llvm::dyn_cast<OperandARM32FlexFpImm>(Src0)) {
+    if (llvm::isa<Variable>(Src0)) {

[Jim Stichnoth, 2016/01/27 22:10:49]

Are vmovss and vmovdd always going to have a Variable as their second argument? 
If so, I would change their declarations from Operand to Variable in that
parameter.

[Karl, 2016/01/27 22:58:09]

Done.

+      Asm->vmovss(Dest, Src0, getPredicate());
+      return;
+    } else if (const auto *FpImm =
+                   llvm::dyn_cast<OperandARM32FlexFpImm>(Src0)) {
       Asm->vmovs(Dest, FpImm, getPredicate());
       return;
     }
-    break;
+    assert(!Asm->needsTextFixup());
+    return;
   case IceType_f64:
-    if (const auto *FpImm = llvm::dyn_cast<OperandARM32FlexFpImm>(Src0)) {
+    if (llvm::isa<Variable>(Src0)) {
+      Asm->vmovdd(Dest, Src0, getPredicate());
+      return;
+    } else if (const auto *FpImm =
+                   llvm::dyn_cast<OperandARM32FlexFpImm>(Src0)) {
       Asm->vmovd(Dest, FpImm, getPredicate());
       return;
     }
-    break;
+    assert(!Asm->needsTextFixup());
+    return;
   }
   // TODO(kschimpf) Handle register to register move.
   Asm->setNeedsTextFixup();
   return;
 }
 
 void InstARM32Mov::emitIASCoreVFPMove(const Cfg *Func) const {
   auto *Asm = Func->getAssembler<ARM32::AssemblerARM32>();
   Operand *Src0 = getSrc(0);
   if (!llvm::isa<Variable>(Src0))
@@ skipping 1172 matching lines @@
 
 template class InstARM32FourAddrGPR<InstARM32::Mla>;
 template class InstARM32FourAddrGPR<InstARM32::Mls>;
 
 template class InstARM32CmpLike<InstARM32::Cmn>;
 template class InstARM32CmpLike<InstARM32::Cmp>;
 template class InstARM32CmpLike<InstARM32::Tst>;
 
 } // end of namespace ARM32
 } // end of namespace Ice