Add vmov between integers and floats in ARM assembler.

src/IceInstARM32.cpp

Index: src/IceInstARM32.cpp
diff --git a/src/IceInstARM32.cpp b/src/IceInstARM32.cpp
index be8488c44e7c4f6e1c898245adb6d977b9290b3c..74b79ec183a54a099a2c8ddf12291a901b474537 100644
--- a/src/IceInstARM32.cpp
+++ b/src/IceInstARM32.cpp
@@ -1207,11 +1207,21 @@ void InstARM32Mov::emitIASCoreVFPMove(const Cfg *Func) const {
   default:
     // TODO(kschimpf): Fill this out more.
     return Asm->setNeedsTextFixup();
+  case IceType_i1:
+  case IceType_i8:
+  case IceType_i16:
+  case IceType_i32:

[Jim Stichnoth, 2016/01/27 22:59:22]

If you're planning on eventually listing all of the IceTypes, you might want to
just "handle" i64 now (presumably as an error).

[Karl, 2016/01/27 23:36:42]

Because of "preprocessing" before this method is called, not all cases will be
covered. I64 for example is not. However, I will add that case in, and a TODO to
consider flattening methods into a single switch.

+    assert(Src0->getType() == IceType_f32 && "Expected int to float move");
+    Asm->vmovrs(Dest, Src0, getPredicate());
+    return;
   case IceType_f32:
     switch (Src0->getType()) {
     default:
-      // TODO(kschimpf): Fill this out more?
-      return Asm->setNeedsTextFixup();
+      assert(false && "Expected float to int move");
+      return;
+    case IceType_i1:
+    case IceType_i8:
+    case IceType_i16:
     case IceType_i32:
       return Asm->vmovsr(Dest, Src0, getPredicate());
     }