Subzero: Remove the GlobalContext::GlobalDeclarations vector.
Elements were added to this vector, but never inspected, so it is
essentially a useless field. Plus, the removal allows us to remove a
couple of friend declarations.
BUG=none
R=kschimpf@google.com
Review URL: https://codereview.chromium.org/814163004
diff --git a/src/IceGlobalInits.h b/src/IceGlobalInits.h
index 082be3c..0d560a1 100644
--- a/src/IceGlobalInits.h
+++ b/src/IceGlobalInits.h
@@ -91,11 +91,9 @@
class FunctionDeclaration : public GlobalDeclaration {
FunctionDeclaration(const FunctionDeclaration &) = delete;
FunctionDeclaration &operator=(const FunctionDeclaration &) = delete;
- friend class GlobalContext;
public:
- static FunctionDeclaration *create(GlobalContext *Ctx,
- const FuncSigType &Signature,
+ static FunctionDeclaration *create(const FuncSigType &Signature,
llvm::CallingConv::ID CallingConv,
llvm::GlobalValue::LinkageTypes Linkage,
bool IsProto);
@@ -129,10 +127,6 @@
class VariableDeclaration : public GlobalDeclaration {
VariableDeclaration(const VariableDeclaration &) = delete;
VariableDeclaration &operator=(const VariableDeclaration &) = delete;
- friend class GlobalContext;
- // TODO(kschimpf) Factor out allocation of initializers into the
- // global context, so that memory allocation/collection can be
- // optimized.
public:
/// Base class for a global variable initializer.
class Initializer {
@@ -248,7 +242,7 @@
/// Models the list of initializers.
typedef std::vector<Initializer *> InitializerListType;
- static VariableDeclaration *create(GlobalContext *Ctx);
+ static VariableDeclaration *create();
~VariableDeclaration() final;
const InitializerListType &getInitializers() const { return Initializers; }
