Subzero: Shorten global prefixes in non-DUMP build.

src/IceBrowserCompileServer.cpp

@@ -172,13 +172,6 @@ void BrowserCompileServer::getParsedFlags(uint32_t NumThreads, int argc,
   Flags->setUseSandboxing(true);
   Flags->setOutFileType(FT_Elf);
   Flags->setTargetArch(getTargetArch());
-  // Make the prefixes short to reduce the chance that string construction will
-  // have to resort to malloc().  Note that this won't change anything if the
-  // pexe was finalize with "--no-strip-syms".  TODO(stichnot): This will be
-  // unnecessary when we stop directly constructing strings that are used for
-  // names and lookup keys.
-  Flags->setDefaultFunctionPrefix("F");
-  Flags->setDefaultGlobalPrefix("G");
   ExtraFlags->setBuildOnRead(true);
   ExtraFlags->setInputFileFormat(llvm::PNaClFormat);
 }

src/IceClFlags.cpp

@@ -84,14 +84,14 @@ cl::opt<std::string>
     DefaultFunctionPrefix("default-function-prefix",
                           cl::desc("Define default function prefix for naming "
                                    "unnamed functions"),
-                          cl::init("Function"));
+                          cl::init(Ice::BuildDefs::dump() ? "Function" : "F"));
 
 /// Define default global prefix for naming unnamed globals.
 cl::opt<std::string>
     DefaultGlobalPrefix("default-global-prefix",
                         cl::desc("Define default global prefix for naming "
                                  "unnamed globals"),
-                        cl::init("Global"));
+                        cl::init(Ice::BuildDefs::dump() ? "Global" : "G"));
 
 /// Disable hybrid assembly when -filetype=iasm.
 cl::opt<bool> DisableHybridAssembly(