diff options
author | Philip Reames <listmail@philipreames.com> | 2014-12-11 01:47:23 +0000 |
---|---|---|
committer | Philip Reames <listmail@philipreames.com> | 2014-12-11 01:47:23 +0000 |
commit | 1e3089749721136482b47233fef8f1d045302e25 (patch) | |
tree | 0965be0f9a3c63a08d3070420368896cd81abd78 /llvm/lib/CodeGen/GCMetadataPrinter.cpp | |
parent | f910e8a04dc862d44349f34d38f11e385aefbea8 (diff) | |
download | llvm-1e3089749721136482b47233fef8f1d045302e25.zip llvm-1e3089749721136482b47233fef8f1d045302e25.tar.gz llvm-1e3089749721136482b47233fef8f1d045302e25.tar.bz2 |
GCStrategy should not own GCFunctionInfo
This change moves the ownership and access of GCFunctionInfo (the object which describes the safepoints associated with a safepoint under GCRoot) to GCModuleInfo. Previously, this was owned by GCStrategy which was in turned owned by GCModuleInfo. This made GCStrategy module specific which is 'surprising' given it's name and other purposes.
There's a few more changes needed, but we're getting towards the point we can reuse GCStrategy for gc.statepoint as well.
p.s. The style of this code ends up being a mess. I was trying to move code around without otherwise changing much. Once I get the ownership structure rearranged, I will go through and fixup spacing, naming, comments etc.
Differential Revision: http://reviews.llvm.org/D6587
llvm-svn: 223994
Diffstat (limited to 'llvm/lib/CodeGen/GCMetadataPrinter.cpp')
-rw-r--r-- | llvm/lib/CodeGen/GCMetadataPrinter.cpp | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/llvm/lib/CodeGen/GCMetadataPrinter.cpp b/llvm/lib/CodeGen/GCMetadataPrinter.cpp index ad238cc..fdff4a7 100644 --- a/llvm/lib/CodeGen/GCMetadataPrinter.cpp +++ b/llvm/lib/CodeGen/GCMetadataPrinter.cpp @@ -17,11 +17,3 @@ using namespace llvm; GCMetadataPrinter::GCMetadataPrinter() { } GCMetadataPrinter::~GCMetadataPrinter() { } - -void GCMetadataPrinter::beginAssembly(Module &M, AsmPrinter &AP) { - // Default is no action. -} - -void GCMetadataPrinter::finishAssembly(Module &M, AsmPrinter &AP) { - // Default is no action. -} |