Add option to force filetype=asm for testing

src/IceTargetLoweringX8664.cpp

 //===- subzero/src/IceTargetLoweringX8664.cpp - x86-64 lowering -----------===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
@@ skipping 349 matching lines @@
 
 void TargetX8664::lowerRet(const InstRet *Inst) {
   Variable *Reg = nullptr;
   if (Inst->hasRetValue()) {
     Operand *Src0 = legalize(Inst->getRetValue());
     if (isVectorType(Src0->getType()) ||
         isScalarFloatingType(Src0->getType())) {
       Reg = legalizeToReg(Src0, Traits::RegisterSet::Reg_xmm0);
     } else {
       assert(isScalarIntegerType(Src0->getType()));
-      _mov(Reg, Src0, Traits::RegisterSet::Reg_eax);
+      if (Src0->getType() == IceType_i8) {

[Jim Stichnoth, 2015/12/20 18:42:23]

Same comment as in x86-32 - handle IceType_i1 as well

[sehr, 2016/01/07 18:53:11]

Done.

+        // TODO(sehr,stichnot): I think these need to sign/zero extend.
+        _mov(Reg, Src0, Traits::RegisterSet::Reg_al);
+      } else if (Src0->getType() == IceType_i16) {
+        _mov(Reg, Src0, Traits::RegisterSet::Reg_ax);
+      } else {
+        _mov(Reg, Src0, Traits::RegisterSet::Reg_eax);

[Jim Stichnoth, 2015/12/20 18:42:23]

You probably need to deal with i32/eax and i64/rax as well, otherwise it will
likely break the x86-64 resurrection CL.

[sehr, 2016/01/07 18:53:11]

John beat me to the punch.  It should work there also.

+      }
     }
   }
   // Add a ret instruction even if sandboxing is enabled, because addEpilog
   // explicitly looks for a ret instruction as a marker for where to insert the
   // frame removal instructions.
   _ret(Reg);
   // Add a fake use of esp to make sure esp stays alive for the entire
   // function. Otherwise post-call esp adjustments get dead-code eliminated.
   keepEspLiveAtExit();
 }
@@ skipping 596 matching lines @@
 // case the high-level table has extra entries.
 #define X(tag, sizeLog2, align, elts, elty, str)                               \
   static_assert(_table1_##tag == _table2_##tag,                                \
                 "Inconsistency between ICETYPEX8664_TABLE and ICETYPE_TABLE");
 ICETYPE_TABLE
 #undef X
 } // end of namespace dummy3
 } // end of anonymous namespace
 
 } // end of namespace Ice