mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-12-13 04:30:23 +00:00
Update global merge pass according to Duncan's advices:
- Remove useless includes - Change misleading comments - Move code into doFinalization git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@177445 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
f36a4afaae
commit
9deb91722c
@ -61,10 +61,8 @@
|
||||
#include "llvm/IR/DerivedTypes.h"
|
||||
#include "llvm/IR/Function.h"
|
||||
#include "llvm/IR/GlobalVariable.h"
|
||||
#include "llvm/IR/InlineAsm.h"
|
||||
#include "llvm/IR/Instructions.h"
|
||||
#include "llvm/IR/Intrinsics.h"
|
||||
#include "llvm/IR/IntrinsicInst.h"
|
||||
#include "llvm/IR/Module.h"
|
||||
#include "llvm/Pass.h"
|
||||
#include "llvm/Support/CommandLine.h"
|
||||
@ -101,7 +99,7 @@ namespace {
|
||||
/// Collect every variables marked as "used"
|
||||
void collectUsedGlobalVariables(Module &M);
|
||||
|
||||
/// Keep track of the GlobalVariable that are marked as "used"
|
||||
/// Keep track of the GlobalVariable that must not be merged away
|
||||
SmallPtrSet<const GlobalVariable *, 16> MustKeepGlobalVariables;
|
||||
|
||||
public:
|
||||
@ -113,6 +111,7 @@ namespace {
|
||||
|
||||
virtual bool doInitialization(Module &M);
|
||||
virtual bool runOnFunction(Function &F);
|
||||
virtual bool doFinalization(Module &M);
|
||||
|
||||
const char *getPassName() const {
|
||||
return "Merge internal globals";
|
||||
@ -211,9 +210,6 @@ void GlobalMerge::collectUsedGlobalVariables(Module &M) {
|
||||
}
|
||||
|
||||
void GlobalMerge::setMustKeepGlobalVariables(Module &M) {
|
||||
// If we already processed a Module, UsedGlobalVariables may have been
|
||||
// populated. Reset the information for this module.
|
||||
MustKeepGlobalVariables.clear();
|
||||
collectUsedGlobalVariables(M);
|
||||
|
||||
for (Module::iterator IFn = M.begin(), IEndFn = M.end(); IFn != IEndFn;
|
||||
@ -268,8 +264,6 @@ bool GlobalMerge::doInitialization(Module &M) {
|
||||
continue;
|
||||
|
||||
// Ignore all "required" globals:
|
||||
// - the ones used for EH
|
||||
// - the ones marked with "used" attribute
|
||||
if (isMustKeepGlobalVariable(I))
|
||||
continue;
|
||||
|
||||
@ -307,6 +301,11 @@ bool GlobalMerge::runOnFunction(Function &F) {
|
||||
return false;
|
||||
}
|
||||
|
||||
bool GlobalMerge::doFinalization(Module &M) {
|
||||
MustKeepGlobalVariables.clear();
|
||||
return false;
|
||||
}
|
||||
|
||||
Pass *llvm::createGlobalMergePass(const TargetLowering *tli) {
|
||||
return new GlobalMerge(tli);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user