Allow ability to name unnamed global addresses in Subzero.

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
 // machine code.
 //
 //===----------------------------------------------------------------------===//
 
 #include "IceTranslator.h"
 
 #include "IceCfg.h"
 #include "IceClFlags.h"
+#include "IceDefs.h"
 #include "IceTargetLowering.h"
+#include "llvm/IR/Module.h"
 
 #include <iostream>
 
 using namespace Ice;
 
 Translator::~Translator() {}
 
+namespace {
+static inline void setValueName(llvm::Value *V, const char *Kind,

[Jim Stichnoth, 2014/09/12 17:20:09]

Don't use static.  And can you omit inline?

[Karl, 2014/09/12 17:41:58]

Done.

+                                const std::string &Prefix, uint32_t &NameIndex,

[Jim Stichnoth, 2014/09/12 17:20:09]

IceString

[Karl, 2014/09/12 17:41:58]

Done.

+                                Ice::Ostream &errs) {

[Jim Stichnoth, 2014/09/12 17:20:09]

Remove Ice:: everywhere in this file (or maybe better, remove "using namespace
Ice;").

[Karl, 2014/09/12 17:41:58]

Done.

+  if (V->hasName()) {
+    const std::string &Name(V->getName());
+    if (Name.find(Prefix) == 0) {
+      errs << "Warning: Default " << Kind << " prefix '" << Prefix
+           << "' conflicts with name '" << Name << "'.\n";
+    }
+    return;
+  }

[Karl, 2014/09/12 17:41:59]

Added check if NameIndex == 0, then use prefix as name instead so that we are
more like pnacl-llc.

+  std::string Buffer;
+  llvm::raw_string_ostream StrBuf(Buffer);
+  StrBuf << Prefix << NameIndex;
+  V->setName(StrBuf.str());
+  ++NameIndex;
+}
+}

[Jim Stichnoth, 2014/09/12 17:20:09]

// end of anonymous namespace

[Karl, 2014/09/12 17:41:59]

Done.

+
+void Translator::nameUnnamedGlobalAddresses(llvm::Module *Mod) {
+  const std::string &GlobalPrefix = Flags.DefaultGlobalPrefix;

[Jim Stichnoth, 2014/09/12 17:20:09]

IceString

[Karl, 2014/09/12 17:41:58]

Done.

+  Ice::Ostream &errs = Ctx->getStrDump();
+  if (!GlobalPrefix.empty()) {
+    uint32_t NameIndex = 0;
+    for (llvm::Module::global_iterator I = Mod->global_begin(),
+                                       E = Mod->global_end();
+         I != E; ++I) {
+      setValueName(I, "global", GlobalPrefix, NameIndex, errs);
+    }
+  }
+  const std::string &FunctionPrefix = Flags.DefaultFunctionPrefix;

[Jim Stichnoth, 2014/09/12 17:20:09]

IceString

[Karl, 2014/09/12 17:41:58]

Done.

+  if (FunctionPrefix.empty())
+    return;
+  uint32_t NameIndex = 0;
+  for (llvm::Module::iterator I = Mod->begin(), E = Mod->end(); I != E; ++I) {
+    setValueName(I, "function", FunctionPrefix, NameIndex, errs);
+  }
+}
+
 void Translator::translateFcn(Ice::Cfg *Fcn) {
   Func.reset(Fcn);
   if (Ctx->getFlags().DisableInternal)
     Func->setInternal(false);
   if (Ctx->getFlags().DisableTranslation) {
     Func->dump();
   } else {
     Ice::Timer TTranslate;
     Func->translate();
     if (Ctx->getFlags().SubzeroTimingEnabled) {
@@ skipping 12 matching lines @@
       std::cerr << "[Subzero timing] Emit function " << Func->getFunctionName()
                 << ": " << TEmit.getElapsedSec() << " sec\n";
     }
   }
 }
 
 void Translator::emitConstants() {
   if (!Ctx->getFlags().DisableTranslation && Func)
     Func->getTarget()->emitConstants();
 }