Introduce the notion of function addresses in Subzero.

src/IceGlobalContext.h

Index: src/IceGlobalContext.h
diff --git a/src/IceGlobalContext.h b/src/IceGlobalContext.h
index cc5da5800ca604f71a239924966a5e8e014679bc..fa6223923547762fd7b156a34db51713011aacf6 100644
--- a/src/IceGlobalContext.h
+++ b/src/IceGlobalContext.h
@@ -21,6 +21,7 @@
 #include "llvm/Support/raw_ostream.h"
 
 #include "IceDefs.h"
+#include "IceClFlags.h"

[Jim Stichnoth, 2014/10/10 13:15:00]

Is this new include necessary?

[Karl, 2014/10/10 20:17:30]

See line 220. On that line it uses Ctx>getFlags().SubzeroTimiingEnabled. Hence,
yes, it is needed (it didn't create a problem because before this CL, all files
that included this include file also included IceClFlags.h).

 #include "IceIntrinsics.h"
 #include "IceRNG.h"
 #include "IceTimerTree.h"
@@ -29,6 +30,9 @@
 namespace Ice {
 
 class ClFlags;
+class FuncSigType;
+class Function;
+class GlobalVariable;
 
 // This class collects rudimentary statistics during translation.
 class CodeStats {
@@ -117,6 +121,15 @@ public:
   // getConstantPool() returns a copy of the constant pool for
   // constants of a given type.
   ConstantList getConstantPool(Type Ty) const;
+  // Returns a new function, allocated in an internal memory pool.
+  // Ownership of the function is maintained by this class instance.
+  Function *newFunction(const FuncSigType *Signature, unsigned CallingConv,
+                        unsigned Linkage, bool IsProto);
+
+  // Returns a new global variable, allocated in an internal memory
+  // pool.  Ownership of the function is maintained by this class
+  // instance.
+  GlobalVariable *newGlobalVariable();
 
   const ClFlags &getFlags() const { return Flags; }
 
@@ -186,6 +199,7 @@ private:
   CodeStats StatsFunction;
   CodeStats StatsCumulative;
   std::vector<TimerStack> Timers;
+  std::vector<GlobalAddress *> GlobalAddresses;
   GlobalContext(const GlobalContext &) = delete;
   GlobalContext &operator=(const GlobalContext &) = delete;