Subzero, Wasm: Dynamically reallocate read buffer. Runtime improvements.

src/IceCompiler.cpp

 //===- subzero/src/IceCompiler.cpp - Driver for bitcode translation -------===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
@@ skipping 38 matching lines @@
 namespace Ice {
 
 namespace {
 
 bool llvmIRInput(const std::string &Filename) {
   return BuildDefs::llvmIrAsInput() &&
          std::regex_match(Filename, std::regex(".*\\.ll"));
 }
 
 bool wasmInput(const std::string &Filename) {
-  return BuildDefs::llvmIrAsInput() &&
-         std::regex_match(Filename, std::regex(".*\\.wasm"));
+  return std::regex_match(Filename, std::regex(".*\\.wasm"));

[Jim Stichnoth, 2016/04/26 22:36:43]

Instead of removing BuildDefs::llvmIrAsInput(), can you replace it with
BuildDefs::wasm() ?

The point here is to avoid pulling in the regex library code if not needed.

[Eric Holk, 2016/04/27 21:45:20]

Done.

 }
 
 } // end of anonymous namespace
 
 void Compiler::run(const Ice::ClFlags &Flags, GlobalContext &Ctx,
                    std::unique_ptr<llvm::DataStreamer> &&InputStream) {
   // The Minimal build (specifically, when dump()/emit() are not implemented)
   // allows only --filetype=obj. Check here to avoid cryptic error messages
   // downstream.
   if (!BuildDefs::dump() && getFlags().getOutFileType() != FT_Elf) {
@@ skipping 94 matching lines @@
     Ctx.dumpTimers();
 
   if (getFlags().getTimeEachFunction()) {
     constexpr bool NoDumpCumulative = false;
     Ctx.dumpTimers(GlobalContext::TSK_Funcs, NoDumpCumulative);
   }
   Ctx.dumpStats();
 }
 
 } // end of namespace Ice