Subzero: Use the linear-scan register allocator for Om1 as well.

src/IceTranslator.cpp

 //===- subzero/src/IceTranslator.cpp - ICE to machine code ------*- C++ -*-===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file defines the general driver class for translating ICE to
@@ skipping 65 matching lines @@
       !matchSymbolName(Func->getFunctionName(),
                        Ctx->getFlags().TranslateOnly)) {
     Func->dump();
   } else {
     Func->translate();
     if (Func->hasError()) {
       std::cerr << "ICE translation error: " << Func->getError() << "\n";
       ErrorStatus = true;
     }
 
-    if (Ctx->getFlags().UseIntegratedAssembler) {

[Jim Stichnoth, 2014/11/14 22:14:33]

Without the !ErrorStatus guard, Om1 was aborting during emission when
encountering an infinite-weight Variable without a register, and the "wrong"
error was causing one of the lit tests to fail.

-      Func->emitIAS();
-    } else {
-      Func->emit();
+    if (!ErrorStatus) {
+      if (Ctx->getFlags().UseIntegratedAssembler) {
+        Func->emitIAS();
+      } else {
+        Func->emit();
+      }
     }
     Ctx->dumpStats(Func->getFunctionName());
   }
 
   Ctx->setVerbose(OldVerboseMask);
 }
 
 void Translator::emitConstants() {
   if (!Ctx->getFlags().DisableTranslation && Func)
     Func->getTarget()->emitConstants();
@@ skipping 10 matching lines @@
   const IceString &TranslateOnly = Ctx->getFlags().TranslateOnly;
   for (const Ice::VariableDeclaration *Global : VariableDeclarations) {
     if (DumpGlobalVariables)
       Global->dump(getContext(), Stream);
     if (!DisableTranslation &&
         matchSymbolName(Global->getName(), TranslateOnly))
       GlobalLowering->lower(*Global);
   }
   GlobalLowering.reset();
 }